-
Notifications
You must be signed in to change notification settings - Fork 207
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
added --include-status
option
#1280
Conversation
Evidencehelp
Execute result
case: not found status
default
|
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1280 +/- ##
==========================================
+ Coverage 81.49% 81.54% +0.04%
==========================================
Files 27 27
Lines 24220 24293 +73
==========================================
+ Hits 19739 19809 +70
- Misses 4481 4484 +3 ☔ View full report in Codecov by Sentry. |
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.
I confirmed that the number of items when test, stable, and experimental are specified in include-status matches the number of items when loaded by default :)
I also confirmed that integration-test was also successful. LGTM!!🚀
https://github.com/Yamato-Security/hayabusa/actions/runs/7902568990/job/21568517716
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.
@hitenkoku LGTM! Thanks so much!
What Changed
--include-status
option