Added support for build number in Github Actions by rand0m86 · Pull Request #228 · git-commit-id/git-commit-id-plugin-core

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@rand0m86

Copy link Copy Markdown

Context

Added support for determining the build number on Github Actions. Apparently it was not that easy task when the initial support was added.

Contributor Checklist

  • Added relevant integration or unit tests to verify the changes
  • Update the Readme or any other documentation (including relevant Javadoc)
  • Ensured that tests pass locally: mvn clean package
  • Ensured that the code meets the current checkstyle coding style definition: mvn clean verify -Pcheckstyle -Dmaven.test.skip=true -B

@TheSnoozer TheSnoozer merged commit 819e54c into git-commit-id:master

May 20, 2025

4 of 24 checks passed

@TheSnoozer TheSnoozer added this to the next milestone

May 20, 2025

@rand0m86

Copy link Copy Markdown

Author

Thanks @TheSnoozer, it was quick 🙂.
Any expected day/month for the next release of the gradle plugin, by any chance?

@TheSnoozer

Copy link Copy Markdown

Contributor

Nothing concrete planned :-) Let me make a release once the tests pass :-)

@TheSnoozer

Copy link Copy Markdown

Contributor

Wow...I haven't done a release for quite some time now. I managed to release the git-commit-id-plugin-core part os version 6.0.0 now. I'll try to release the maven plugin with the relevant changes by the weekend.

rand0m86 reacted with thumbs up emoji

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TheSnoozer TheSnoozer TheSnoozer left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

6.0.0

Development

Successfully merging this pull request may close these issues.

3 participants

@rand0m86 @TheSnoozer @mahasv