-
Notifications
You must be signed in to change notification settings - Fork 343
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
Use "avocado-latest-release" repository #6057
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6057 +/- ##
==========================================
- Coverage 54.48% 54.46% -0.03%
==========================================
Files 202 202
Lines 21885 21890 +5
==========================================
- Hits 11925 11922 -3
- Misses 9960 9968 +8 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
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.
Hi @clebergnu, changes LGTM, thanks. Maybe could you please also update our next release notes in https://github.com/avocado-framework/avocado/blob/master/docs/source/releases/next.rst to not forget to mention this change during the release? What do you think?
In both the installation instructions, and in the release guide. This reflects changes in distros and the availability of features such as Fedora (and EPEL) modules. Fixes: avocado-framework#6044 Reference: https://fedoraproject.org/wiki/Changes/RetireModularity Signed-off-by: Cleber Rosa <[email protected]>
0be736c
to
ae82a4d
Compare
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.
Hi @clebergnu, thanks for the update. I have one more comment to release notes. Please have a look.
* The installation of Avocado from distribution's packages, changed | ||
when compared to the previous version due to changes in repositories | ||
and support for "package modules" in some distributions. Please | ||
refer to the installation guide for updated instructions. |
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.
Since you are referencing to the installation guide here, I would add a ref:
here to the new installation instructions. IMO, it would be easier for readers to find the changes.
In both the installation instructions, and in the release guide.
This reflects changes in distros and the availability of features such as Fedora (and EPEL) modules.
Fixes: #6044
Reference: https://fedoraproject.org/wiki/Changes/RetireModularity