-
Notifications
You must be signed in to change notification settings - Fork 22
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
Merge ign-fuel-tools7 ➡️ gz-fuel-tools8 #401
Conversation
* Use on `push` only on stable branches to avoid duplicate runs * Update project automation Signed-off-by: Addisu Z. Taddese <[email protected]>
* Use on `push` only on stable branches to avoid duplicate runs * Update project automation Signed-off-by: Addisu Z. Taddese <[email protected]>
Signed-off-by: Michael Carroll <[email protected]>
Signed-off-by: Michael Carroll <[email protected]>
Signed-off-by: Addisu Z. Taddese <[email protected]>
Merge ign-fuel-tools4 ➡️ ign-fuel-tools7
Signed-off-by: Addisu Z. Taddese <[email protected]>
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.
The PR looks very innocent, is CI supposed to fail?
The failing test is calling |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## gz-fuel-tools8 #401 +/- ##
===============================================
Coverage 75.32% 75.32%
===============================================
Files 20 20
Lines 2910 2910
===============================================
Hits 2192 2192
Misses 718 718 ☔ View full report in Codecov by Sentry. |
➡️ Forward port
Port
ign-fuel-tools7
➡️gz-fuel-tools8
Branch comparison: gz-fuel-tools8...ign-fuel-tools7
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)