Skip to content

link to CI status dashboard#7098

Merged
MichaelChirico merged 1 commit into
masterfrom
CI-status
Jun 26, 2025
Merged

link to CI status dashboard#7098
MichaelChirico merged 1 commit into
masterfrom
CI-status

Conversation

@jangorecki
Copy link
Copy Markdown
Member

@jangorecki jangorecki commented Jun 26, 2025

Thanks for addressing minor issues with CI (updating allowed NOTEs count, fixing compilation warnings, etc.)!
As a result the dashboard is useful again!

I am proposing to change the link for our CI status dashboard rather than CI pipeline list.

Why to change?

  • dashboard is much more useful overview than a pipelines list

Why not to change?

  • status icon (green/red) is out of sync for the dashboard content
    • if the status icon is red, then it means that dashboard haven't been updated at all, so any information available there refers to latest successful pipeline.
    • if the status icon is green, then dashboard have been updated, but failed might be still present there (as is now).

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 26, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.69%. Comparing base (e32e553) to head (e5a5ed9).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #7098   +/-   ##
=======================================
  Coverage   98.69%   98.69%           
=======================================
  Files          79       79           
  Lines       14680    14680           
=======================================
  Hits        14489    14489           
  Misses        191      191           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Member

@MichaelChirico MichaelChirico left a comment

Choose a reason for hiding this comment

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

LGTM, agree this is an improvement

@MichaelChirico MichaelChirico merged commit 8e02858 into master Jun 26, 2025
11 checks passed
@MichaelChirico MichaelChirico deleted the CI-status branch June 26, 2025 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants