Skip to content

Conversation

@bhirsz
Copy link
Member

@bhirsz bhirsz commented Nov 14, 2020

Closes #198

@bhirsz bhirsz requested a review from mnojek as a code owner November 14, 2020 16:44


class MissingTrailingBlankLineChecker(VisitorChecker):
class EmptyLinesChecker(VisitorChecker):
Copy link
Member Author

Choose a reason for hiding this comment

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

Figured out this name would be more suitable

Copy link
Member

Choose a reason for hiding this comment

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

Sure it is!

@codecov-io
Copy link

codecov-io commented Nov 14, 2020

Codecov Report

Merging #230 (f650999) into master (6144cd7) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #230   +/-   ##
=======================================
  Coverage   99.05%   99.05%           
=======================================
  Files          33       33           
  Lines        2110     2124   +14     
=======================================
+ Hits         2090     2104   +14     
  Misses         20       20           
Impacted Files Coverage Δ
robocop/checkers/spacing.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6144cd7...f650999. Read the comment docs.



class MissingTrailingBlankLineChecker(VisitorChecker):
class EmptyLinesChecker(VisitorChecker):
Copy link
Member

Choose a reason for hiding this comment

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

Sure it is!

Co-authored-by: Mateusz Nojek <[email protected]>
@bhirsz bhirsz merged commit 18b9700 into master Nov 15, 2020
@bhirsz bhirsz deleted the feature/invalid_empty_lines_after_section branch November 15, 2020 20:41
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.

Rule: Number of empty lines after section *** Header ***

4 participants