Skip to content

Fix baseline commit to current one#2337

Merged
wojtekn merged 1 commit intotrunkfrom
fix/baseline-commit-for-metrics-tests
Dec 30, 2025
Merged

Fix baseline commit to current one#2337
wojtekn merged 1 commit intotrunkfrom
fix/baseline-commit-for-metrics-tests

Conversation

@wojtekn
Copy link
Copy Markdown
Contributor

@wojtekn wojtekn commented Dec 30, 2025

Related issues

Proposed Changes

  • I propose fixing the baseline commit used in performance tests. I used the incorrect, older one in tests migrated to the Buildkite.

Testing Instructions

  • Confirm build goes through

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

@wojtekn wojtekn requested a review from a team December 30, 2025 10:45
@wojtekn wojtekn self-assigned this Dec 30, 2025
@wpmobilebot
Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing 9d52185 vs trunk

site-editor

Metric trunk 9d52185 Diff Change
load 7667.00 ms 5060.00 ms -2607.00 ms 🟢 -34.0%

site-startup

Metric trunk 9d52185 Diff Change
siteCreation 9101.00 ms 9086.00 ms -15.00 ms 🟢 -0.2%
siteStartup 3953.00 ms 3958.00 ms +5.00 ms 🔴 0.1%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change

Copy link
Copy Markdown
Contributor

@nightnei nightnei left a comment

Choose a reason for hiding this comment

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

LGTM
Checked - indeed this commit was used before

@wojtekn wojtekn merged commit bb80ab7 into trunk Dec 30, 2025
9 checks passed
@wojtekn wojtekn deleted the fix/baseline-commit-for-metrics-tests branch December 30, 2025 11:36
@wojtekn
Copy link
Copy Markdown
Contributor Author

wojtekn commented Dec 30, 2025

The metrics job worked successfully in the trunk now, and submitted results to CodeVitals:

CleanShot 2025-12-30 at 13 08 23@2x

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