Skip to content

Conversation

benank
Copy link
Contributor

@benank benank commented Jul 29, 2022

Adds patch ordering to assessment and installation patch summaries.

Patches are ordered by the following criteria:

  • First by classification: Security, Critical, Other, Unclassified
  • Then by patchInstallationState (for install operations): Failed, Installed, Available, Pending, Excluded, NotSelected

@benank benank requested review from a team, kjohn-msft and rane-rajasi and removed request for a team July 29, 2022 18:34
@codecov
Copy link

codecov bot commented Jul 29, 2022

Codecov Report

Merging #158 (9960ca2) into master (255687b) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #158      +/-   ##
==========================================
+ Coverage   87.93%   87.98%   +0.04%     
==========================================
  Files          86       86              
  Lines       12013    12058      +45     
==========================================
+ Hits        10564    10609      +45     
  Misses       1449     1449              
Flag Coverage Δ
python27 86.34% <100.00%> (+0.07%) ⬆️
python39 87.95% <100.00%> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/core/src/bootstrap/Constants.py 98.32% <100.00%> (+0.01%) ⬆️
src/core/src/service_interfaces/StatusHandler.py 92.51% <100.00%> (+0.28%) ⬆️
src/core/tests/Test_CoreMain.py 99.85% <100.00%> (ø)
src/core/tests/Test_StatusHandler.py 99.66% <100.00%> (+0.03%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@benank benank changed the title Add patch ordering to assessment and installation summaries Add patch ordering to assessment and installation patch summaries Jul 29, 2022
@benank benank requested a review from rane-rajasi August 16, 2022 17:22
@kjohn-msft kjohn-msft merged commit 4653433 into master Aug 24, 2022
@benank benank deleted the bankiel-patch-order branch August 24, 2022 20:45
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