Skip to content
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

Add alert and error fields to output (same as for pre-conversion) #4

Merged
merged 3 commits into from
Nov 21, 2023

Conversation

andywaltlova
Copy link
Collaborator

@andywaltlova andywaltlova commented Nov 16, 2023

Updated tests because of slight refactor
Fix Process error outside of try block

Signed-off-by: Andrea Waltlova <[email protected]>
@andywaltlova andywaltlova mentioned this pull request Nov 16, 2023
2 tasks
@andywaltlova andywaltlova force-pushed the update-preupgrade-task-reporting branch from bc13f08 to 6e53a72 Compare November 16, 2023 10:02
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1f427e2) 100.00% compared to head (6e53a72) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #4   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines          135       145   +10     
=========================================
+ Hits           135       145   +10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@Monnte Monnte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested, works good, just little changes needed. Really like that the report is reordered that the inhibitor and problem count message is at the bottom 👍

scripts/leapp_preupgrade.py Outdated Show resolved Hide resolved
scripts/leapp_preupgrade.py Show resolved Hide resolved
scripts/leapp_preupgrade.py Outdated Show resolved Hide resolved
scripts/leapp_preupgrade.py Outdated Show resolved Hide resolved
@andywaltlova andywaltlova force-pushed the update-preupgrade-task-reporting branch from f7f4ab5 to edad91c Compare November 16, 2023 21:44
@andywaltlova andywaltlova requested a review from Monnte November 21, 2023 08:21
@andywaltlova andywaltlova merged commit d89edef into main Nov 21, 2023
1 check passed
@andywaltlova andywaltlova deleted the update-preupgrade-task-reporting branch January 17, 2024 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants