Skip to content

Add default tag#520

Merged
camillobruni merged 3 commits into
WebKit:mainfrom
camillobruni:2025-07-09_default_tag
Jul 9, 2025
Merged

Add default tag#520
camillobruni merged 3 commits into
WebKit:mainfrom
camillobruni:2025-07-09_default_tag

Conversation

@camillobruni
Copy link
Copy Markdown
Contributor

@camillobruni camillobruni commented Jul 9, 2025

  • Add "default" tag for all default-scored workloads (as an explicit distinction from "experimental")
  • Replace .disabled with .enabled to simplify a bit more code

Drive-by-fix:

  • Rename _validateSuiteRessultsStub to _validateSuiteResultsStub

@netlify
Copy link
Copy Markdown

netlify Bot commented Jul 9, 2025

Deploy Preview for webkit-speedometer-preview ready!

Name Link
🔨 Latest commit b8cb83d
🔍 Latest deploy log https://app.netlify.com/projects/webkit-speedometer-preview/deploys/686e7988d7956e0008a4707a
😎 Deploy Preview https://deploy-preview-520--webkit-speedometer-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@camillobruni camillobruni added the trivial change A change that doesn't affect benchmark results label Jul 9, 2025
Copy link
Copy Markdown
Contributor

@bgrins bgrins left a comment

Choose a reason for hiding this comment

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

I like the idea here. Means things are off by default unless they have the explicit tag

@camillobruni camillobruni merged commit deba93c into WebKit:main Jul 9, 2025
8 checks passed
@camillobruni camillobruni deleted the 2025-07-09_default_tag branch July 9, 2025 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

trivial change A change that doesn't affect benchmark results

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants