Add option to keep pass statements #143
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes in this PR:
- ignore-pass-statements: allows the user to opt out of the removal of useless pass statements if desired.
- ignore-pass-after-docstring: allows the user to keep pass statements after a docstring (see below for caveat).
Caveat: since autoflake does not use an AST, deciding whether a pass statement follows a docstring is not really possible (at least not with a LOT of effort), so the approach is to keep pass statements right after a line ending with triple quotes. This is most probably in 99% of the cases a pass after a docstring.
This adds the possibility of keeping pass statements after docstrings which are not useless and is one of the requested features in the repo, see #73, #10. I know that autoflake will already keep the pass statement if:
Closes #9.
Closes #10.
Closes #73.