-
-
Notifications
You must be signed in to change notification settings - Fork 301
feat: pre-release seperated changelog #454
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
feat: pre-release seperated changelog #454
Conversation
I'd appreciate it if this behavior could be configured in the .cz.toml
file. Is it so?
I'd appreciate it if this behavior could be configured in the
.cz.toml
file. Is it so?
Hi, well no. It would be very nice indeed to have this as config keys. As you see I opened this pr over a year ago and didn't progressed with it since I don't know how (read 'Additional Context' from the pr description) I was guessing that this change isn't welcome or that the maintainers do not have the time to help in such situations. My changes are not unit tested just "use tested". And if I am completely honest I am not eager to do this atm, since I 1. don't rly like python and 2. Therefore have a suboptimal to no python dev setup and no knowledge how to write the tests how to execute them (like layed out in Additional Context)
But this said the implementation, while being untested seemed rather complete to me. I would love to see that someone takes it over since at this point after 1 year of no reply I probably know less about my own code than the maintainers - since they at least know the language and the environment
This is a failed attempt by me. I am not good enough with python. The feature was now implemented by proper devs :')
Their implementation does not cover everything but it is a start -better than the mess that i created
Uh oh!
There was an error while loading. Please reload this page.
Description
Adds 2 parameters :
--separate-pre-changelog
: use a separate changelog for each type of pre-release--separate-pre-releases
: changes in the changelog will always relate to the latest release of the same pre-release type(and their respective implementation)
Checklist
./script/format
and./script/test
locally to ensure this change passes linter check and testExpected behavior
When
--separate-pre-releases
is used entries (features and fixes) should appear in a release even if they technicaly belong to a pre releaseWhen
--separate-pre-changelog
is used a changelog for each pre-release type should exist and be maintained by cz.Steps to Test This Pull Request
(
cz
being my modified version)Expected:
The CHANGELOG.md should look like this:
Additional context
This issue is kind of a draft, as i found no guidance of how to contribute.
While the PR template tells me what to do i do now know how. I cannot execute the above testcases as when i run
python3 __main__.py
the imports use my local installation of comittizen (usingvenv
and also not using venv) instead of my modified code. I also do not rly know what how to write or execute test cases. do i need to remove my local installation and do something like pip install my repo?I would love to contribute hight quality (well tested and documented etc) but as there is no documentation of how to contribute, I am a little stuck. (i followed the instructions in
docs/contributing.md
)If you would be so kind to provide some basic information and feedback i would check all the requirements.
(Some of this issues might be related to my very little development experience in python as i never dealt with package management etc. but I think a person should be able to contribute without greater knowledge about the eco-system)
Adresses issues