Skip to content

Conversation

@TheSnoozer
Copy link
Collaborator

Hi konrad,
it's me once again ;)
I extended the describe-option of the git-commit-id-plugin with the match-Option (@see: #97).

Any suggestions for improvements are welcome.

BTW: I updated the README a bit...change it if you don't like the description ;)

@TheSnoozer
Copy link
Collaborator Author

Hi @ktoso,
can u review this pls?

Thx :)

@ktoso ktoso force-pushed the master branch 7 times, most recently from 97e085d to 7ee11ec Compare October 21, 2014 22:05
@ktoso
Copy link
Collaborator

ktoso commented Dec 7, 2014

hey, I'm back with some time to look into this. I want to release something tomorrow so I'll review and possibly merge (even if manual) :-)

Copy link
Collaborator

Choose a reason for hiding this comment

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

thanks!

@ktoso ktoso merged commit 60cb978 into git-commit-id:master Dec 7, 2014
@ktoso
Copy link
Collaborator

ktoso commented Dec 7, 2014

Merged!
Thanks a lot @TheSnoozer, and sorry for the delays.
Planned release time is tomorrow night CET :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

nice, thanks!

@TheSnoozer
Copy link
Collaborator Author

Hi @ktoso !
Thanks for reviewing.
And no worries about the delay ;)

Thanks for this great plugin :)

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.

2 participants