-
Notifications
You must be signed in to change notification settings - Fork 15
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
Retire native ubipop publishing [RHELDST-25072] #243
Retire native ubipop publishing [RHELDST-25072] #243
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #243 +/- ##
==========================================
- Coverage 98.77% 98.70% -0.07%
==========================================
Files 7 6 -1
Lines 735 696 -39
==========================================
- Hits 726 687 -39
Misses 9 9 ☔ View full report in Codecov by Sentry. |
dda36b1
to
ba22d9f
Compare
run tests |
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.
Almost OK, please rebase and resolve conflicts (it should likely fix the lock file)
ba22d9f
to
7513884
Compare
Update lock file with a separate PR, based on the recent pyproject.toml changes merged in PR release-engineering#242, which caused conflict in PR release-engineering#243. This PR is solely meant for rebasing.
7513884
to
eb35d85
Compare
run tests |
@Gdetrane |
+ Remove `cdn.py` module containing the `Publisher` class + Remove the *UBIPOP_SKIP_PUBLISH* env variable used for switching between publishers + Removed the *publisher* option for classes using it
eb35d85
to
0702979
Compare
@rbikar should be okay now |
run tests |
@Gdetrane |
cdn.py
module containing thePublisher
class