Skip to content

ci: upload test results in merge queue to main context#792

Merged
colinaaa merged 1 commit intolynx-family:mainfrom
colinaaa:colin/0513/workflow
May 13, 2025
Merged

ci: upload test results in merge queue to main context#792
colinaaa merged 1 commit intolynx-family:mainfrom
colinaaa:colin/0513/workflow

Conversation

@colinaaa
Copy link
Copy Markdown
Collaborator

Summary

Fix a regression of #646. The event name should be merge_group instead of merge_queue. See: https://docs.github.com/en/actions/writing-workflows/choosing-when-your-workflow-runs/events-that-trigger-workflows#merge_group.

Also fix an issue that publish and build-all share the same concurrency.group.

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented May 13, 2025

⚠️ No Changeset found

Latest commit: 1f09e54

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@codecov
Copy link
Copy Markdown

codecov bot commented May 13, 2025

Bundle Report

Changes will increase total bundle size by 3.35kB (0.52%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
@lynx-js/example-react-lynx-cjs 321.14kB 3.35kB (1.05%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: @lynx-js/example-react-lynx-cjs

Assets Changed:

Asset Name Size Change Total Size Change (%)
main.lynx.bundle 1.74kB 89.87kB 1.97%
.rspeedy/main/main-thread.js 744 bytes 40.8kB 1.86%
.rspeedy/main/background.*.js 864 bytes 39.91kB 2.21%

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented May 13, 2025

CodSpeed Performance Report

Merging #792 will not alter performance

Comparing colinaaa:colin/0513/workflow (1f09e54) with main (ee03212)

Summary

✅ 2 untouched benchmarks

@colinaaa colinaaa marked this pull request as ready for review May 13, 2025 11:05
@colinaaa colinaaa requested a review from PupilTong May 13, 2025 11:05
@colinaaa colinaaa added this pull request to the merge queue May 13, 2025
Merged via the queue into lynx-family:main with commit 5795869 May 13, 2025
38 checks passed
@colinaaa colinaaa deleted the colin/0513/workflow branch May 13, 2025 13:34
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.

2 participants