Skip to content

Add uknown checker status to guidelinestatistics - #4975

Merged
bruntib merged 10 commits into
Ericsson:masterfrom
dkrupp:guideline_stat_refactor
Jul 30, 2026
Merged

Add uknown checker status to guidelinestatistics#4975
bruntib merged 10 commits into
Ericsson:masterfrom
dkrupp:guideline_stat_refactor

Conversation

@dkrupp

@dkrupp dkrupp commented Jul 17, 2026

Copy link
Copy Markdown
Member

-make the guideline statistics query faster and simpler -show the guideline statistics even if there are some runs with uknown status -remove unnecessary API calls from the guideline statistics vue page

@dkrupp dkrupp added the WIP 💣 Work In Progress label Jul 17, 2026
@dkrupp
dkrupp force-pushed the guideline_stat_refactor branch 8 times, most recently from 165857e to c42b871 Compare July 21, 2026 14:01
@dkrupp dkrupp added bug 🐛 bugfix 🔨 and removed WIP 💣 Work In Progress labels Jul 22, 2026
Comment thread web/server/vue-cli/src/components/Statistics/Guideline/GuidelineStatistics.vue Outdated
Comment thread web/server/codechecker_server/api/report_server.py Outdated
Comment thread web/server/codechecker_server/api/report_server.py
Comment thread web/server/vue-cli/src/components/Statistics/Guideline/GuidelineStatistics.vue Outdated
dkrupp added 7 commits July 29, 2026 11:11
-make the guideline statistics query faster and simpler
-show the guideline statistics even if there are some runs with uknown status
-remove unnecessary API calls from the guideline statistics vue page
-simplify the countChips component display
-add a question mark icon to show the runs with uknown status
@dkrupp
dkrupp force-pushed the guideline_stat_refactor branch from 43a7f5e to 29cb0c1 Compare July 29, 2026 11:15
@dkrupp
dkrupp requested a review from bruntib July 29, 2026 14:17
@dkrupp

dkrupp commented Jul 29, 2026

Copy link
Copy Markdown
Member Author

All remarks fixed, all tests passing. @bruntib Please check if all is good now. thx.

Checker.id, CheckerSetItem.enabled)
)

LOG.info(checker_status_query)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please, remove this log.

@bruntib
bruntib merged commit 794904c into Ericsson:master Jul 30, 2026
17 of 18 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.

2 participants