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

chore: fix package.json version on release #45

Merged

Conversation

feloy
Copy link
Contributor

@feloy feloy commented Feb 16, 2024

Fixes #44

Comment on lines 59 to 60
sed -i "s#version\":\ \"\(.*\)\",#version\":\ \"${{ steps.TAG_UTIL.outputs.extVersion }}\",#g" package.json
git add package.json
Copy link

Choose a reason for hiding this comment

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

I've not deeply looked but it looks like we're executing the commands at the root of the repository, package.json of the extension is inside the podman-desktop-extension folder so I'm wondering if it's not podman-desktop-extension/package.json

for the sed and the git add instruction

Copy link
Member

Choose a reason for hiding this comment

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

Seems there is an issue here

@feloy feloy requested a review from benoitf February 16, 2024 10:19
Copy link
Member

@jeffmaury jeffmaury left a comment

Choose a reason for hiding this comment

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

LGTM

@feloy feloy merged commit b5f5590 into redhat-developer:main Feb 16, 2024
1 check passed
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.

published releases does not have proper versioning
3 participants