Skip to content

Remove 'martinbeentjes/npm-get-version-action' from release workflow#887

Merged
benmccann merged 1 commit intomapbox:masterfrom
WifiDB:remove_unsupported_action
Dec 7, 2024
Merged

Remove 'martinbeentjes/npm-get-version-action' from release workflow#887
benmccann merged 1 commit intomapbox:masterfrom
WifiDB:remove_unsupported_action

Conversation

@acalcutt
Copy link
Copy Markdown
Contributor

@acalcutt acalcutt commented Dec 7, 2024

Removes the 'martinbeentjes/npm-get-version-action' from the release workflow.

I am already getting this value in the first job, so I just added that to the outputs and used that variable where 'steps.package-version.outputs.current-version' was used before.

I tested this in my fork and it looks like it pulls the version as expected
https://github.com/WifiDB/node-pre-gyp/actions/runs/12215279116
https://github.com/WifiDB/node-pre-gyp/releases/tag/v2.0.0-rc.3
https://www.npmjs.com/package/@acalcutt/node-pre-gyp-test/v/2.0.0-rc.3

@acalcutt acalcutt requested a review from a team as a code owner December 7, 2024 18:50
@acalcutt acalcutt mentioned this pull request Dec 7, 2024
@benmccann benmccann merged commit e59c5e2 into mapbox:master Dec 7, 2024
@acalcutt
Copy link
Copy Markdown
Contributor Author

acalcutt commented Dec 7, 2024

Nice, at least the workflow ran this time. It saw that 1.0.11 was already published, so it didn't actually publish anything yet....but at least it ran.

image

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