Skip to content
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

permeable-1 #1

Open
wants to merge 29 commits into
base: master
Choose a base branch
from
Open

permeable-1 #1

wants to merge 29 commits into from

Conversation

hellochunqiu
Copy link

更新收藏

Yuyang Wang and others added 29 commits June 21, 2018 23:24
This one does Python but we could add other languages.

Travis CI can be used to do automated testing on all pull requests before they are reviewed and merged. This service is free to open source projects like this one. The Continuous Integration service must be turned on by @w1109790800 logging into https://travis-ci.com/w1109790800 with GitHub credentials and turning on the switch for this repo.

This config will have http://flake8.pycqa.org flag the build if there are Python syntax errors or undefined names.

E901,E999,F821,F822,F823 are the "showstopper" flake8 issues that can halt the runtime with a SyntaxError, NameError, etc. Most other flake8 issues are merely "style violations" -- useful for readability but they do not effect runtime safety. This PR therefore recommends a flake8 run of these tests on the entire codebase.

    F821: undefined name name
    F822: undefined name name in __all__
    F823: local variable name referenced before assignment
    E901: SyntaxError or IndentationError
    E999: SyntaxError -- failed to compile a file into an Abstract Syntax Tree
Run some automated testing...
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.

1 participant