Skip to content

Fix Travis-CI build error. #858

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

reyoung
Copy link
Collaborator

@reyoung reyoung commented Dec 13, 2016

* error because merge of PaddlePaddle#711. The old issue don't check pre-commit
  hooks.
@reyoung reyoung requested a review from gangliao December 13, 2016 07:51
except AssertionError as e:
self.logger.error(
def __call__(self, obj, filename):
for items in self.generator(obj, filename):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Python初学者好奇:这个PR修复的问题是由错误的indentation导致的呀?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

是因为目前Paddle在持续集成的Travis-CI上面检查了代码风格。原理是直接用yapf format python code,用clang-format格式化cpp code,如果有diff,那么就直接CI报错。

由于这个检查是最近一些PR加进去的,之前的PR并没有强制的跑这个检查。所以出错了。

解决办法就是直接重新format一遍就好了。命令是pre-commit run -a

@reyoung reyoung requested a review from hedaoyuan December 13, 2016 08:37
@reyoung reyoung merged commit 1c26ce2 into PaddlePaddle:develop Dec 13, 2016
@reyoung reyoung deleted the feature/fix_travis_build_error branch December 13, 2016 09:39
wangxicoding pushed a commit to wangxicoding/Paddle that referenced this pull request Dec 9, 2021
[Feat] Add custom op for FasterTransformer decoder
yaozhixin pushed a commit to graphcore/Paddle-fork that referenced this pull request Jul 1, 2022
…-ipu_8845a1cf6a51b4fc3c80641ce03e049d3aadc487

[AUTO PR] Pulling [develop] into [develop-ipu] [2022-07-01 03:05:17+08:00]
lizexu123 pushed a commit to lizexu123/Paddle that referenced this pull request Feb 23, 2024
zhangyuqin1998 pushed a commit to zhangyuqin1998/Paddle that referenced this pull request Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants