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

PyLint compliance, resolves #21 #24

Closed
wants to merge 1 commit into from

Conversation

louis-r
Copy link
Collaborator

@louis-r louis-r commented Oct 5, 2017

Why this change was necessary:

  • Make sure we adopt a standard style

This change addresses the need by:

  • Adding tests .py files to check in .travis_yaml.

Potential side-effects:

Why this change was necessary:

* Make sure we adopt a standard style

This change addresses the need by:

* Adding tests .py files to check in  .travis_yaml.

Potential side-effects:

*
@louis-r
Copy link
Collaborator Author

louis-r commented Oct 5, 2017

@Edouard360 Make sure to adopt Google Style doc in PyCharm :)

@louis-r
Copy link
Collaborator Author

louis-r commented Oct 5, 2017

@Edouard360 Note that the pylint_checks.txt is a temporary solution, when all files will be compliant it will not be needed anymore.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 55.856% when pulling 1637473 on louis-r:louisremus into fe1e101 on Edouard360:master.

@louis-r louis-r closed this Oct 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants