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

feat(frontend): add analyzer score summary #2735

Merged
merged 7 commits into from Jun 16, 2023
Merged

Conversation

jorgeepc
Copy link
Contributor

@jorgeepc jorgeepc commented Jun 14, 2023

This PR adds the Analyzer Score to the summary section of Tests and TestRuns.

Changes

  • analyzer score in the TestSummary response from the BE
  • add analyzer score to the list of Tests
  • add analyzer score to the list of TestRuns

Fixes

Checklist

  • tested locally
  • added new dependencies
  • updated the docs
  • added a test

Loom

https://www.loom.com/share/7df37c53a58c40f5824b0fd4f9b43903?sid=0f286df0-cb86-45e7-8987-b81e2ea7ff99

@jorgeepc jorgeepc self-assigned this Jun 14, 2023
@jorgeepc jorgeepc marked this pull request as ready for review June 14, 2023 22:05
@jorgeepc jorgeepc requested a review from xoscar June 14, 2023 22:06
Base automatically changed from 2647-analyzer-ui-improvements to main June 14, 2023 22:10
Copy link
Collaborator

@xoscar xoscar left a comment

Choose a reason for hiding this comment

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

Great job @jorgeepc the redirect behavior is a great addition!

@olha23
Copy link

olha23 commented Jun 15, 2023

Thank you @jorgeepc looks good. The only small comment - can you make the same difference between score, last time result and last run time

@olha23
Copy link

olha23 commented Jun 15, 2023

Because now it’s different:
Screenshot 2023-06-15 at 10 17 42 AM

@jorgeepc jorgeepc merged commit 65a7fa4 into main Jun 16, 2023
24 checks passed
@jorgeepc jorgeepc deleted the 2650-add-analyzer-summary branch June 16, 2023 21:16
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.

[Analyzer Results] Showing the high level results
3 participants