Skip to content
This repository has been archived by the owner. It is now read-only.

Add pylint and fix all errors #238

Merged
merged 1 commit into from Feb 1, 2019
Merged

Add pylint and fix all errors #238

merged 1 commit into from Feb 1, 2019

Conversation

@c-w
Copy link
Contributor

@c-w c-w commented Feb 1, 2019

In #235 it became apparent that full docstring coverage is desired for the project. As such, this change adds pylint to the CI and fixes all the lint errors. Among others, pylint ensures that all methods, classes and modules have comments.

agogosml/Makefile Show resolved Hide resolved
sayar
sayar approved these changes Feb 1, 2019
@c-w c-w merged commit 1b13ef7 into master Feb 1, 2019
3 checks passed
@c-w c-w deleted the pylint branch Feb 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants