-
Notifications
You must be signed in to change notification settings - Fork 22
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
GitHub action for automatizing shup release with openWebStart configu… #2232
base: develop
Are you sure you want to change the base?
Conversation
…ration .jnlp file
The build will fail because the maven jar signer will need a keystore to do its job.
For now as tested on a local branch :
2 minor issues :
|
branches: [ "master" ] | ||
paths: | ||
- 'shanoir-uploader/**' | ||
pull_request: |
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.
Hi @LaurentPV, I wonder, if we should do something on each pull request or only, when something goes on master? I think even with each new integration into master, we will not release a new version of ShUp. I think it would be interesting for each push into master to create a new "timestamp" in RELEASE_DATE on ShUpConfig.
I think the process of having a new version, as today the tags for the prod release by Anthony are driven by a human decision, not by an integration on master. With kind regards, Michael
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.
Ok, I will add a new commit modifying the RELEASE_DATE constant with the timestamp of the push on master branch.
Do you mean that the tag of the release should be the release_date instead of the release_version ?
It seems weird not to have the tag of a version linked to a release but in the same time it would be more convenient because it assures the uniqueness of each release tag. For now if you forget to modify the version in the pom.xml before pushing into master the release will fail because the previous release exists already.
…release timestamp.
Creation of shup.jnlp configuration file for OpenWebStart that points towards .jnlp file and .jar archive located in releases/tag/${project.version} of git repository (when git release action will be completed). In this .jnlp file we inject the value associated with project.version in shanoir-uploader pom.xml so that when releasing a new version we only need to modify the version in pom.xml
Adding in shanoir-uploader pom.xml the copy of .jnlp file from ${project.basedir} to ${project.build.directory}, we set the maven antrun plugin copies as "prepare-package" phase to be sure that all files are available in /target directory for next steps.
Adding a maven replacer plugin step to inject the ${project.version} value of the pom.xml file into the .jnlp file located in /target directory
Creation of release_shup.yml file in .github/workflows to configure the GitHub action of releasing shanoir-uploader with the right project.version value and the .jnlp and .jar files in the same release every time there is a push onto the master branch concerning the content of /shanoir-uploader directory.