Skip to content
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

Automate shorthand major version tag updating #28

Merged
merged 4 commits into from
Jun 23, 2021

Conversation

rekmarks
Copy link
Member

@rekmarks rekmarks commented Jun 23, 2021

Automate the updating of the shorthand major version tag (e.g. v1) at the end of the local publish-release workflow. We accomplish this by adding an output to this action named release-version.

@rekmarks rekmarks requested a review from a team as a code owner June 23, 2021 18:29
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- name: Update shorthand major version tag
run: ./scripts/update-major-version-tag.sh ${{ steps.publish-release.outputs.RELEASE_VERSION }}
Copy link
Member

Choose a reason for hiding this comment

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

I don't see RELEASE_VERSION being set as an output 🤔

Copy link
Member Author

@rekmarks rekmarks Jun 23, 2021

Choose a reason for hiding this comment

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

Good point. It's set in the get-release-version step of action.yml. I'm not sure if it's set explicitly in the action or not (see here), but I did it anyway in bc83f11.

Copy link
Member

Choose a reason for hiding this comment

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

Huh. So those docs don't seem to clarify whether environment variables set are automatically considered as "outputs". That's the part I wasn't sure of. Was this shown to work before you made that change? The other shorthand major version tag updating PR still doesn't set the output explicitly.

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

I can confirm that the output works as expected, although I found another issue with my implementation in this PR that I address here: #30

@rekmarks rekmarks force-pushed the automate-shorthand-major-version-tag-update branch from 97afa4b to bc83f11 Compare June 23, 2021 19:59
@rekmarks rekmarks requested a review from Gudahtt June 23, 2021 20:09
@rekmarks rekmarks merged commit 038147a into main Jun 23, 2021
@rekmarks rekmarks deleted the automate-shorthand-major-version-tag-update branch June 23, 2021 21:37
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