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

fix: remove blank lines after line beginning with 'def' #171

Merged
merged 7 commits into from
Apr 12, 2023
Merged

fix: remove blank lines after line beginning with 'def' #171

merged 7 commits into from
Apr 12, 2023

Conversation

weibullguy
Copy link
Member

@weibullguy weibullguy commented Apr 4, 2023 •
edited
Loading

Fixes problem with blank lines between methods/functions being removed when the last line in the first function began with "def".

Closes #154
Closes #156
Closes #165
Closes #172
Closes #173

@weibullguy weibullguy added P: bug PEP 257 violation or existing functionality that doesn't work as documented C: convention Relates to docstring format convention V: patch Bumps the patch version labels Apr 4, 2023
@coveralls
Copy link

Pull Request Test Coverage Report for Build 4607599825

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 0.0%
Totals Coverage Status
Change from base Build 4602880828: 0.0%
Covered Lines: 0
Relevant Lines: 0
💛 - Coveralls

1 similar comment
@coveralls
Copy link

Pull Request Test Coverage Report for Build 4607599825

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 0.0%
Totals Coverage Status
Change from base Build 4602880828: 0.0%
Covered Lines: 0
Relevant Lines: 0
💛 - Coveralls

@weibullguy weibullguy merged commit f1b0fa6 into master Apr 12, 2023
@weibullguy weibullguy deleted the fix/remove_lines_after_default branch April 21, 2023 18:40
@weibullguy weibullguy added S: merged Closed with work merged to repository and removed V: patch Bumps the patch version labels Apr 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
C: convention Relates to docstring format convention P: bug PEP 257 violation or existing functionality that doesn't work as documented S: merged Closed with work merged to repository
Projects
None yet
Milestone
No milestone
2 participants
@weibullguy @coveralls