Travis CI: Add tests to find undefined names #325
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
你好!感谢你反馈的问题/bug，但是你的描述好像是空的，我们需要你完整的信息，这样才能帮你解决问题 如果不知道怎么写，在新建issue的时候有若干个模板可供选择，祝好！ |
|
Thanks for your contribution! I noticed that the build failed due to an obsolete module "Themis" of our project, I wonder if there's a way to add a config file for flake8 to ignore the whole "themis" directory? |
hhyo
approved these changes
Jul 15, 2019
|
@hhyo 我好像没权限, 这个pr不知道怎么回事没有拿到覆盖率, 加个未定义变量应该问题不大, 应该可以合了, 后期看看应该可以加更多的静态代码检查 LGTM, but it seems I dont have permission to merge this since no coverage report been found, I've already requested the owner of project to deal with this. |
xxlaila
pushed a commit
to xxlaila/Archery
that referenced
this pull request
Feb 15, 2025
Travis CI: Add tests to find undefined names
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.