Skip to content
New issue

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

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

Already on GitHub? # to your account

get version from package.json instead of release #1053

Merged
merged 1 commit into from
Feb 13, 2024
Merged

Conversation

elrayle
Copy link
Collaborator

@elrayle elrayle commented Feb 13, 2024

If the action is triggered by a release, ${{ github.event.release.tag_name }} has the version. If we manually run this action manually, ${{ github.event.release.tag_name }} is empty.

To always have the version for tagging the created docker image and to pass to the webapp, this PR gets the version directly from the package.json file.

NOTE: The package.json file must have the correct version when running manually.

If the action is triggered by a release, `${{ github.event.release.tag_name }}` has the version.  If we manually run this action manually, `${{ github.event.release.tag_name }}` is empty.

To always have the version for tagging the created docker image and to pass to the webapp, this PR gets the version directly from the package.json file.

_NOTE: The package.json file must have the correct version when running manually._
@elrayle elrayle merged commit 4065288 into prod Feb 13, 2024
1 check passed
@elrayle elrayle deleted the elr/prod-get-version branch February 13, 2024 21:43
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants