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.

Sign up for GitHub

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

Jump to bottom

autoflake in setup.cfg #68

Closed
wants to merge 1 commit into from
Closed

autoflake in setup.cfg #68

wants to merge 1 commit into from

Conversation

TylerYep
Copy link

@TylerYep TylerYep commented May 25, 2020 •
edited
Loading

Adds support for setup.cfg arguments (under the [autoflake] section)

I will add tests after the initial idea is reviewed!

@coveralls
Copy link

coveralls commented May 25, 2020 •
edited
Loading

Coverage Status

Coverage decreased (-0.8%) to 98.664% when pulling 38b8fa5 on TylerYep:master into b330072 on myint:master.

myint
myint requested changes Aug 23, 2020
View reviewed changes
Copy link
Member

@myint myint left a comment

Choose a reason for hiding this comment

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

Seems like a useful feature to me. Though, it would be good if it followed pycodestyle's convention of looking for the setup.cfg file relative to the file arguments rather than the working directory.

@akeeman akeeman mentioned this pull request Sep 25, 2020
Merged
3 tasks
@TylerYep
Copy link
Author

#79 will implement this functionality in a better way.

@TylerYep TylerYep closed this May 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@myint myint myint requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@TylerYep @coveralls @myint