-
Notifications
You must be signed in to change notification settings - Fork 10
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
Change default upper section from "header" to "condensed_header" #209
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #209 +/- ##
==========================================
- Coverage 98.36% 98.33% -0.04%
==========================================
Files 358 358
Lines 28831 28831
==========================================
- Hits 28360 28351 -9
- Misses 471 480 +9
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Codecov Report
@@ Coverage Diff @@
## main #209 +/- ##
==========================================
- Coverage 98.36% 98.33% -0.04%
==========================================
Files 358 358
Lines 28831 28831
==========================================
- Hits 28360 28351 -9
- Misses 471 480 +9
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Codecov Report
Changes have been made to critical files, which contain lines commonly executed in production. Learn more Additional details and impacted files@@ Coverage Diff @@
## main #209 +/- ##
==========================================
- Coverage 98.32% 98.30% -0.03%
==========================================
Files 389 389
Lines 29527 29527
==========================================
- Hits 29033 29026 -7
- Misses 494 501 +7
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 3 files with indirect coverage changes
|
3f863fb
to
e756de2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but left small comment
@@ -209,7 +209,7 @@ def get_upper_section_names(self, settings): | |||
sections = list(map(lambda l: l.strip(), (settings["layout"] or "").split(","))) | |||
headers = ["newheader", "header", "condensed_header"] | |||
if all(not x in sections for x in headers): | |||
sections.insert(0, "header") | |||
sections.insert(0, "newheader") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"condensed_header" is the new name for "newheader" given that "newheader" will not be "new" if it's the default.
NIT comment being use "condensed_header" as the value, not "newheader"
2e4d4e4
to
e449c0f
Compare
e449c0f
to
5779656
Compare
No description provided.