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

Add tox configuration to the repository #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

he0s
Copy link

@he0s he0s commented Apr 16, 2019

Add tox configuration file, that can be used to run different kind
of tests like syntax test, any custom tests and so forge.

This configuration has some check ignored, the list of them is the following:

N801 # Should be fixed (e.g.: class name 'Graph_UUID_Status' should use CapWords convention)
N802 # Should be fixed (e.g.: function name 'ByID' should be lowercase)
N803 # Should be fixed (e.g.: argument name 'ID' should be lowercase)
N805 # Should be fixed (e.g.: first argument of a method should be named 'self')
N806 # Should be fixed (e.g.: variable 'ID' in function should be lowercase)
N815 # Should be fixed (e.g.: variable '_lib_byID' in class scope should not be mixedCase)
N816 # Should be fixed (e.g.: variable 'streamJS' in global scope should not be mixedCase)

Some checks were skipped by adding # noqa keyword to the code.

All the changes have passed the flake8 tests as well as running unit tests from test.py script.

@he0s he0s changed the title Add tox configuration to the directory Add tox configuration to the repository Apr 16, 2019
Add tox configuration file, that can be used to run different kind
of tests like syntax test, any custom tests and so forge.
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