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

📝 👷 docs: ci: Document and update syncronizing lockfile with release #1047

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

LogFlames
Copy link
Member

See #1042.

The project itself currently runs maven-lockfile directly using jbang in the relase action. Some more thought has to be put into if/how the maven-lockfile/action.yml should/could be used instead. It would be more convenient as it automatically set up the environment needed with java, maven and jbang. But during the release action it must be able to be run on newly created (not yet pushed) branches, on updated pom.xml's that are not commited, etc.

@LogFlames
Copy link
Member Author

Just realised a better idea that to use the action with jbang is to assume mvn is setup for the release (as it is probably needed to build it, and to run mvn version:set e.g.) and then run the plugin directly with mvn io.github.chains-project:maven-lockfile:5.3.5 generate. Easier to do without interfacing with maven-lockfile-github-plugin.

@LogFlames LogFlames changed the title 📝 Document syncronizing lockfile with release 📝 👷 Document and update syncronizing lockfile with release Dec 29, 2024
@LogFlames LogFlames changed the title 📝 👷 Document and update syncronizing lockfile with release 📝 👷 docs: ci: Document and update syncronizing lockfile with release Dec 30, 2024
@LogFlames LogFlames force-pushed the 1042-document-potential-issue-where-lockfile-for-tagged-release-commit-contains-snapshot-version branch from 097c2a3 to 8b4f797 Compare December 30, 2024 01:47
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.

Document potential issue where lockfile for tagged release commit contains -SNAPSHOT version
1 participant