Skip to content

ci: fix version output interpolation in make-version workflow #4220

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 28, 2025

Conversation

dreamorosi
Copy link
Contributor

Summary

Changes

Please provide a summary of what's being changed

This PR updates the make-version.yml workflow, and specifically the step that updates the packages/commons/src/version.ts file with the new version. Some of the refactors done in previous PRs since the last release broke the command by removing enclosing quotes around the version number.

This PR addresses this by simplifying the interpolation, and hopefully fixing the workflow once and for all.

Please add the issue number below, if no issue is present the PR might get blocked and not be reviewed

Issue number: closes #4214


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

@dreamorosi dreamorosi requested a review from svozza July 28, 2025 07:01
@dreamorosi dreamorosi self-assigned this Jul 28, 2025
@pull-request-size pull-request-size bot added the size/XS PR between 0-9 LOC label Jul 28, 2025
@boring-cyborg boring-cyborg bot added the automation This item relates to automation label Jul 28, 2025
Copy link

@dreamorosi dreamorosi merged commit fea274f into main Jul 28, 2025
54 checks passed
@dreamorosi dreamorosi deleted the ci-bump_shared_actions branch July 28, 2025 08:57
@dreamorosi dreamorosi restored the ci-bump_shared_actions branch July 29, 2025 12:01
@dreamorosi dreamorosi deleted the ci-bump_shared_actions branch July 31, 2025 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation This item relates to automation size/XS PR between 0-9 LOC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Maintenance: fix make version workflow
2 participants