FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Final results should be alphabetically sorted by kmiller68 · Pull Request #253 · WebKit/JetStream · GitHub

/ JetStream Public

Final results should be alphabetically sorted - #253

Open
kmiller68 wants to merge 1 commit into
WebKit:mainfrom
kmiller68:tests-should-end-in-sorted-order
ConversationCommits (1)ChecksFiles changed
Open

Final results should be alphabetically sorted#253
kmiller68 wants to merge 1 commit into
WebKit:mainfrom
kmiller68:tests-should-end-in-sorted-order

Conversation

kmiller68 commented Jan 13, 2026

Copy link
Copy Markdown
Contributor

Right now the results end in a reverse alphabetically sorted order. This looks weird. Ideally we would just change the order that we run tests but I worry that will change the results in a non-trivial way (although I didn't try directly).

In future JetStream versions we should revert this and sort the tests in the other direction.

Final results should be alphabetically sorted
1b3f6bd 
Right now the results end in a reverse alphabetically sorted order. This
looks weird. Ideally we would just change the order that we run tests
but I worry that will change the results in a non-trivial way (although
I didn't try directly).

In future JetStream versions we should revert this and sort the tests in
the other direction.

netlify Bot commented Jan 13, 2026 •
edited
Loading

Copy link
Copy Markdown

✅ Deploy Preview for webkit-jetstream-preview ready!

Name Link
🔨 Latest commit 1b3f6bd
🔍 Latest deploy log https://app.netlify.com/projects/webkit-jetstream-preview/deploys/6966cd1367605f00087f7ff1
😎 Deploy Preview https://deploy-preview-253--webkit-jetstream-preview.netlify.app
📱 Preview on mobile Toggle QR Code...



Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

danleh commented Jan 14, 2026

Copy link
Copy Markdown
Contributor

I'd still be happy to change the order of execution to alphabetical. If this causes a big change in score for a particular line-item on any engine, let's investigate. (Part of the discussion around iteration sizes was to not be so dependent, e.g., on GC timing, so if we still are, I'd rather know it.)

camillobruni commented Jan 14, 2026

Copy link
Copy Markdown
Contributor

+1 to changing the underlying order
-1 to having a mismatch between UI and execution order. I always found it very confusing to follow the execution order in the old UI (looking a line item, it's visually moving around while this is not the case currently anymore). The current version makes this very clear (and actually does expose the "weird" underlying reverse execution order)

kmiller68 commented Jan 14, 2026

Copy link
Copy Markdown
Contributor Author

Ok I'll give changing the order a shot and see what happens

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL