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

Reduce size of pep8speaks config file #2201

Closed
matthewturk opened this issue Mar 8, 2019 · 3 comments
Closed

Reduce size of pep8speaks config file #2201

matthewturk opened this issue Mar 8, 2019 · 3 comments
Labels
infrastructure Related to CI, versioning, websites, organizational issues, etc new contributor friendly Good for new contributors!

Comments

@matthewturk
Copy link
Member

After pep8speaks-org/pep8speaks#106 has been merged, it looks like we can reduce the config file presence -- and reduce duplication -- for pep8speaks.

I believe it would be sufficient to remove our .pep8speaks.yml file, but we should investigate if we can remove the "ignore" and "exclude" sections and leave the bits where we define how the bot should talk.

@matthewturk matthewturk added new contributor friendly Good for new contributors! infrastructure Related to CI, versioning, websites, organizational issues, etc labels Mar 8, 2019
@brittonsmith
Copy link
Member

Looks like the PR has now been merged. Someone can give this a try if they like.

@matthewturk
Copy link
Member Author

@brittonsmith Excellent! I think this would be a great first-time contribution...

@neutrinoceros
Copy link
Member

Closing as we stopped using pep8speak for a little while now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Related to CI, versioning, websites, organizational issues, etc new contributor friendly Good for new contributors!
Projects
None yet
Development

No branches or pull requests

3 participants