Skip to content

Add a way to specify files to ignore#41

Closed
thiblahute wants to merge 1 commit intosebdah:masterfrom
thiblahute:master
Closed

Add a way to specify files to ignore#41
thiblahute wants to merge 1 commit intosebdah:masterfrom
thiblahute:master

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 :)

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