-
Notifications
You must be signed in to change notification settings - Fork 96
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Add support for RHEL 8 and CentOS 8 #92
Conversation
siebrand
commented
Oct 18, 2019
- Add key for EPEL-8.
- Update tests for os version 8.
- Add author to contributors.
- Update testing notes.
- Update alternatives.
spec/classes/shared_base.rb
Outdated
|
||
it do | ||
is_expected.to contain_yumrepo('epel').with( | ||
mirrorlist: 'https://mirrors.fedoraproject.org/metalink?repo=epel-8&arch=$basearch', |
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.
You have an extra two spaces of indent here.
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.
Fixed.
spec/classes/shared_base.rb
Outdated
mirrorlist: 'https://mirrors.fedoraproject.org/metalink?repo=epel-7&arch=$basearch', | ||
gpgkey: 'file:///etc/pki/rpm-gpg/RPM-GPG-KEY-EPEL-7', | ||
descr: 'Extra Packages for Enterprise Linux 7 - $basearch' | ||
mirrorlist: 'https://mirrors.fedoraproject.org/metalink?repo=epel-7&arch=$basearch', |
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.
…and an extra two spaces of indent here, as well as many other places.
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.
All cases fixed.
- Add key for EPEL-8. - Update tests for os version 8. - Add author to contributors. - Update testing notes. - Update alternatives.
c6fb790
to
37bd12a
Compare
Are there any updates on the status of this PR? |
Thank you @siebrand! |