-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Simplify release process by generating artifacts in the CI #131
base: master
Are you sure you want to change the base?
Conversation
40e9ad9
to
686183f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested in a different repo and it works apart from the filename being wrong (see inline comment).
.github/workflows/maven.yml
Outdated
asset_name: openrefine-commons-$VERSION_STRING.zip | ||
asset_content_type: application/zip | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Superfluous newline.
.github/workflows/maven.yml
Outdated
with: | ||
upload_url: ${{ steps.get_release_upload_url.outputs.upload_url }} | ||
asset_path: ./target/openrefine-commons.zip | ||
asset_name: openrefine-commons-$VERSION_STRING.zip |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replace value with: openrefine-commons-${{ env.VERSION_STRING }}.zip
.
Apparently using bash env variables doesn't work here. This will just call the file
openrefine-commons-VERSION_STRING.zip. See the difference in the releases. "test" is this version and "test2" is with the proposed change.
When @sebastian-berlin-wmse did the last release I observed that the process was quite complicated. I think it's worth simplifying, so this is a small step towards that, inspired by the setup we have in the OpenRefine repo. It removes the need to generate the release artifact and upload it to the GitHub release.
I haven't tested it so I hope it works well for the next release.