-
Notifications
You must be signed in to change notification settings - Fork 23
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
RHIROS-1302 enable group sorting #294
RHIROS-1302 enable group sorting #294
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #294 +/- ##
==========================================
+ Coverage 20.33% 20.35% +0.02%
==========================================
Files 47 47
Lines 954 953 -1
Branches 179 179
==========================================
Hits 194 194
+ Misses 712 711 -1
Partials 48 48 ☔ View full report in Codecov by Sentry. |
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.
LGTM!
@yash2189 can you test this PR? |
cca3aa8
to
8b5623d
Compare
8b5623d
to
7fdd5df
Compare
@r14chandra I pushed few changes after your review, can you please have a look into PR again? |
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.
Looks good!
Enable group sorting 💥
Jira: https://issues.redhat.com/browse/RHIROS-1302
Why do we need this change? 💭
Please include the context of this change here.
Documentation requires update? 📝
Security Checklist 🔒
Upon raising this PR please go through RedHatInsights/secure-coding-checklist
💂♂️ Checklist 🎯
Additional 📣
Feel free to add any other relevant details such as links, notes, screenshots, here.