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

RHIROS-1302 enable group sorting #294

Merged

Conversation

PreetiW
Copy link
Contributor

@PreetiW PreetiW commented Nov 17, 2023

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? 📝

  • Yes
  • No

Security Checklist 🔒

Upon raising this PR please go through RedHatInsights/secure-coding-checklist

💂‍♂️ Checklist 🎯

  • Bugfix
  • New Feature
  • Refactor
  • Unittests Added
  • DRY code
  • Dependency Added

Additional 📣

Feel free to add any other relevant details such as links, notes, screenshots, here.

@codecov-commenter
Copy link

codecov-commenter commented Nov 17, 2023

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (e378de7) 20.33% compared to head (34ed7ae) 20.35%.

❗ Current head 34ed7ae differs from pull request most recent head c290eb4. Consider uploading reports for the commit c290eb4 to get more accurate results

Files Patch % Lines
src/constants.js 0.00% 1 Missing ⚠️
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.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@r14chandra r14chandra left a comment

Choose a reason for hiding this comment

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

LGTM!

@r14chandra
Copy link
Contributor

@yash2189 can you test this PR?

@PreetiW PreetiW force-pushed the RHIROS-1302-enable-group-sorting branch from cca3aa8 to 8b5623d Compare November 28, 2023 10:56
@PreetiW PreetiW force-pushed the RHIROS-1302-enable-group-sorting branch from 8b5623d to 7fdd5df Compare November 28, 2023 11:37
@PreetiW PreetiW requested a review from r14chandra November 28, 2023 11:44
@PreetiW
Copy link
Contributor Author

PreetiW commented Nov 28, 2023

@r14chandra I pushed few changes after your review, can you please have a look into PR again?

Copy link
Contributor

@r14chandra r14chandra left a comment

Choose a reason for hiding this comment

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

Looks good!

@PreetiW PreetiW merged commit fd14d68 into RedHatInsights:main Dec 8, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants