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

Add bootstrap option to set a prefix for the commit messages #1350

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DriesVerachtert
Copy link

Flux bootstrap already has an option to set a suffix for the commit messages. This change also adds a prefix. It's handy in environments where commit messages have to start with some ticket number for example.

I'm not a go developer: I basically copied the code of the 'commitMessageAppendix' option.

Signed-off-by: Dries Verachtert dries.verachtert@dries.eu

Signed-off-by: Dries Verachtert <dries.verachtert@dries.eu>
@DriesVerachtert
Copy link
Author

I've added the Signed-off-by in the commit message.

Copy link
Member

@hiddeco hiddeco left a comment

Choose a reason for hiding this comment

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

Thank you for submitting a PR 🍰

I think that if there is a need to be able to control both the prefix and the appendix of the Git commit message, it would probably be better to deprecate the -appendix flag in favor of one that allows you to configure a full Git commit message template. Given this would prevent a wild-growth of commit message related flags that all modify different parts of the message.

@olivercp3
Copy link

olivercp3 commented Dec 21, 2022

hello, so how can i set a prefix for the commit messages only for bootstrap now

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.

None yet

3 participants