-
Notifications
You must be signed in to change notification settings - Fork 278
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
[RPM M1] Add distribution supported components list so only specific scripts can take -d parameter #1652
[RPM M1] Add distribution supported components list so only specific scripts can take -d parameter #1652
Conversation
…an take parameter Signed-off-by: Peter Zhu <[email protected]>
Signed-off-by: Peter Zhu <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #1652 +/- ##
=========================================
Coverage 94.68% 94.69%
Complexity 14 14
=========================================
Files 163 163
Lines 3446 3447 +1
Branches 21 21
=========================================
+ Hits 3263 3264 +1
Misses 180 180
Partials 3 3
Continue to review full report at Codecov.
|
It fixes issues coming from #1629. |
This is a hack and I dislike it very much. I propose we go through the pain of adding support to |
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'm ok with the hack but not checking by name.
Signed-off-by: Peter Zhu [email protected]
Description
Add distribution supported components list so only specific scripts can take -d parameter
Issues Resolved
#1651
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.