Skip to content

Conversation

@aaronmedina-dev
Copy link
Contributor

Description of the proposed changes

  • Correctly reference environment variables set via using ${{ env.VAR }} instead of Bash-style $VAR, which is not available across steps. This fixes an issue where the release step failed due to undefined PACKAGE and VERSION variables.

Screenshots (if applicable)

Other solutions considered (if any)

Notes to PR author

⚠️ Please make sure the changes adhere to the guidelines mentioned here

Notes to reviewers

🛈 When you've finished leaving feedback, please add a final comment to the PR tagging the author, letting them know that you have finished leaving feedback

Correctly reference environment variables set via  using ${{ env.VAR }}
instead of Bash-style $VAR, which is not available across steps. This fixes an issue
where the release step failed due to undefined PACKAGE and VERSION variables.
@TheOrangePuff
Copy link
Member

@aaronmedina-dev I'm not sure that this is the issue 🤔 We just pushed through a different release and that worked fine: https://github.com/aligent/cdk-constructs/actions/runs/14458156741

@aaronmedina-dev aaronmedina-dev deleted the chore/release_workflow_fix branch April 15, 2025 01:12
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