Skip to content

feat: add a run-id output - #243

Open
moha-bekh wants to merge 3 commits into
mainfrom
cod-3661-expose-the-run-id-as-an-output-of-the-runner-and-the-github
Open

moha-bekh wants to merge 3 commits into
mainfrom
cod-3661-expose-the-run-id-as-an-output-of-the-runner-and-the-github

Conversation

@moha-bekh

Copy link
Copy Markdown
Member

Add a run-id output with the id of the CodSpeed run the results were uploaded to, so later workflow steps can read the run back.

Depends on CodSpeedHQ/codspeed#551, which writes the id to $GITHUB_OUTPUT. Keep as draft until that ships in a runner release; the "v5.4.0" requirement in action.yml and the README must match that version.

Refs COD-3661

moha-bekh and others added 3 commits September 28, 2026 06:16
Expose the id of the CodSpeed run the results were uploaded to, so that
later steps of the workflow can read the run back.

Refs COD-3661
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The runner only writes the `run-id` step output when
`CODSPEED_EXPORT_RUN_ID` is set. Pass it as an env var rather than a flag
so that older pinned runners ignore it instead of failing.

Refs COD-3661
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@moha-bekh
moha-bekh marked this pull request as ready for review September 29, 2026 12:39
@moha-bekh

Copy link
Copy Markdown
Member Author

Ready for review; merge only after runner v5.4.0 is released and the runner version is bumped.

@GuillaumeLagrange GuillaumeLagrange left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's keep use the env variable only

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