Skip to content

Junos: add empty lines#9236

Merged
dhalperi merged 1 commit intomasterfrom
spr/master/484c5319
Oct 7, 2024
Merged

Junos: add empty lines#9236
dhalperi merged 1 commit intomasterfrom
spr/master/484c5319

Conversation

@dhalperi
Copy link
Copy Markdown
Member

@dhalperi dhalperi commented Oct 7, 2024

Avoids flattening issues when there's a bare semicolon after a comment.

Avoids flattening issues when there's a bare semicolon after a comment.

commit-id:484c5319
@dhalperi dhalperi requested a review from SLarkworthy October 7, 2024 22:43
@batfish-bot
Copy link
Copy Markdown

This change is Reviewable

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.72%. Comparing base (24e1b8d) to head (e6da691).
Report is 56 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9236      +/-   ##
==========================================
- Coverage   72.73%   72.72%   -0.01%     
==========================================
  Files        3314     3314              
  Lines      170019   170019              
  Branches    20048    20048              
==========================================
- Hits       123657   123643      -14     
- Misses      37199    37211      +12     
- Partials     9163     9165       +2     

see 4 files with indirect coverage changes

Copy link
Copy Markdown
Contributor

@SLarkworthy SLarkworthy left a comment

Choose a reason for hiding this comment

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

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @dhalperi)

@dhalperi dhalperi merged commit ea50a77 into master Oct 7, 2024
@dhalperi dhalperi deleted the spr/master/484c5319 branch October 7, 2024 23:00
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.

3 participants