Skip to content

Junos: parse and todo bgp-output-queue-priority#9194

Merged
dhalperi merged 1 commit intomasterfrom
spr/master/90ae5211
Sep 11, 2024
Merged

Junos: parse and todo bgp-output-queue-priority#9194
dhalperi merged 1 commit intomasterfrom
spr/master/90ae5211

Conversation

@dhalperi
Copy link
Copy Markdown
Member

@dhalperi dhalperi commented Sep 11, 2024

Stack:

⚠️ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

@batfish-bot
Copy link
Copy Markdown

This change is Reviewable

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.13%. Comparing base (8b4f410) to head (68e5a39).
Report is 67 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9194      +/-   ##
==========================================
- Coverage   78.13%   78.13%   -0.01%     
==========================================
  Files        3313     3313              
  Lines      169930   169932       +2     
  Branches    20039    20039              
==========================================
- Hits       132783   132772      -11     
- Misses      37147    37160      +13     
Files with missing lines Coverage Δ
...fish/grammar/flatjuniper/ConfigurationBuilder.java 74.71% <100.00%> (+0.01%) ⬆️

... and 2 files with indirect coverage changes

Copy link
Copy Markdown
Contributor

@progwriter progwriter left a comment

Choose a reason for hiding this comment

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

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

@dhalperi dhalperi merged commit 7045fb8 into master Sep 11, 2024
@dhalperi dhalperi deleted the spr/master/90ae5211 branch September 11, 2024 17:46
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