Skip to content

Conversation

@meyer9
Copy link
Collaborator

@meyer9 meyer9 commented Nov 4, 2025

Description

Sort charts on the chart page by moving the most important metrics first.

Testing

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Nov 4, 2025

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

tobidae-cb
tobidae-cb previously approved these changes Nov 4, 2025
@meyer9 meyer9 force-pushed the meyer9/sort-charts branch from 604642c to e8f71f3 Compare November 4, 2025 19:50
@meyer9 meyer9 requested a review from tobidae-cb November 4, 2025 19:50
@cb-heimdall cb-heimdall dismissed tobidae-cb’s stale review November 4, 2025 19:50

Approved review 3418400433 from tobidae-cb is now dismissed due to new commit. Re-request for approval.

@meyer9
Copy link
Collaborator Author

meyer9 commented Nov 4, 2025

sorry, had to rebase

@meyer9 meyer9 enabled auto-merge (squash) November 4, 2025 19:51
@meyer9 meyer9 merged commit b201129 into main Nov 4, 2025
10 checks passed
@meyer9 meyer9 deleted the meyer9/sort-charts branch November 4, 2025 20:07
tobidae-cb pushed a commit that referenced this pull request Dec 5, 2025
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.

4 participants