-
Notifications
You must be signed in to change notification settings - Fork 51
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
Enhance description of sync_changelog
#861
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Nikola Forró <[email protected]>
Build succeeded. ✔️ pre-commit SUCCESS in 1m 27s |
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.
thanks! But I am thinking about whether we shouldn't change this behaviour, it doesn't really align with the option name (we would need to check who is using this option).
|
+1. It is a bit misleading, even with the documentation change. Why do projects, like tmt, don't use packit for generating changelog? |
Yes, this would make the most sense to me. Any ideas how to call it? |
Let's discuss this on the next architecture meeting. @martinhoyer in case you would like to participate, let me know and I'll invite you. |
Thanks, I probably don't have much to add due to the lack of experience with packit (and in general), so all I have to say is in the above comment. To recap:
|
No description provided.