Skip to content

feat: sort charts - #126

Merged
meyer9 merged 1 commit into
mainfrom
meyer9/sort-charts
Nov 4, 2025
Merged

feat: sort charts#126
meyer9 merged 1 commit into
mainfrom
meyer9/sort-charts

Conversation

@meyer9

@meyer9 meyer9 commented Nov 4, 2025

Copy link
Copy Markdown
Collaborator

Description

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

Testing

@cb-heimdall

cb-heimdall commented Nov 4, 2025

Copy link
Copy Markdown
Collaborator

✅ 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

meyer9 commented Nov 4, 2025

Copy link
Copy Markdown
Collaborator Author

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.

3 participants