-
Notifications
You must be signed in to change notification settings - Fork 22
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
Initial draft commit for importance widget component #485
Initial draft commit for importance widget component #485
Conversation
7765607
to
657e8c1
Compare
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.
Besides cleaning up some comments, the code looks good. Haven't tested on my own yet.
Attached is a video of the widget Screenshare.-.2024-03-28.12_21_00.PM.mp4 |
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.
Generally LGTM, great work! There is only 1 small thing that we can fix before merging it
DRAFT for a widget to show test results filtered by component and grouped by test importance