-
Notifications
You must be signed in to change notification settings - Fork 377
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
Implement Fedora disto #2642
Implement Fedora disto #2642
Conversation
Fedora distribution is not covered by Redhat*OS* classes, create a separate one for it. Note: this patch was previosly carried out-of-tree, fedora packages already have it included. Signed-off-by: Vitaly Kuznetsov <vkuznets@redhat.com>
Codecov Report
@@ Coverage Diff @@
## develop #2642 +/- ##
===========================================
- Coverage 72.06% 71.98% -0.08%
===========================================
Files 103 104 +1
Lines 15702 15748 +46
Branches 2237 2241 +4
===========================================
+ Hits 11315 11336 +21
- Misses 3872 3896 +24
- Partials 515 516 +1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Ping |
@narrieta can you take a look at this PR? Would be a great starting point for Fedora support. |
@vittyvk Sorry, for the delay. LGTM, will do one automation run and merge when it completes. You tested your changes on your side, correct? Currently we do not have the infrastructure to test Fedora. |
@narrieta Yes, Fedora was carrying these patches for some time already. I'm smoke-testing every new Fedora build (I'm the package maintainer). |
@vittyvk Thank you. Merged. |
Fedora distribution is not covered by RedhatOS classes, create
a separate one for it.
Note: this patch was previosly carried out-of-tree, fedora packages
already have it included.
Signed-off-by: Vitaly Kuznetsov vkuznets@redhat.com
Description
Issue #
PR information
Quality of Code and Contribution Guidelines