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 a way to specify files to ignore #41

Closed
wants to merge 1 commit into from

Conversation

thiblahute
Copy link

This way we can iteratively pylint files in a big project ensuring that no pylint regression happens meanwhile.

@thiblahute
Copy link
Author

Build failure does not seem to be related to that commit at all tbh.

@thiblahute
Copy link
Author

Hello there,

Is that project maintained or should I use a fork? I really need that commit so we can start using linting incrementally :)

@@ -87,7 +95,7 @@ def _parse_score(pylint_output):

def check_repo(
limit, pylint='pylint', pylintrc='.pylintrc', pylint_params=None,
suppress_report=False):
suppress_report=False, ignored_files=[]):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Function arguments' default value should not be set to a list, it being mutable.

sebdah added a commit that referenced this pull request Sep 8, 2016
Due to conflict I manually implemented #41.
@sebdah
Copy link
Owner

sebdah commented Sep 8, 2016

Manually merged and now released in 2.2.0.

@sebdah sebdah closed this Sep 8, 2016
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.

4 participants