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

chore: update changelog #202

Merged
merged 1 commit into from
May 28, 2024
Merged

chore: update changelog #202

merged 1 commit into from
May 28, 2024

Conversation

nixos-asia-bot[bot]
Copy link
Contributor

No description provided.

@github-actions github-actions bot enabled auto-merge (rebase) May 28, 2024 11:32
@github-actions github-actions bot merged commit e48f790 into main May 28, 2024
4 checks passed
@srid
Copy link
Member

srid commented May 30, 2024

Personally, I feel like doing this manually as part of the release process would be better (to avoid commit history clutter mainly). And link "Unreleased" like https://github.com/stanislav-tkach/os_info/blob/master/CHANGELOG.md does.

But this is just a subjective preference, so feel free to adopt whatever process is most suitable for managing this repo. @shivaraj-bh

@shivaraj-bh shivaraj-bh deleted the update_changelog branch May 31, 2024 06:39
@shivaraj-bh
Copy link
Member

Personally, I feel like doing this manually as part of the release process would be better (to avoid commit history clutter mainly)

Yeah, fair enough. I just wanted to see if this works, so that I can write an article about this. I will just observe if it works as expected, when it gets triggered this Sunday and I will disable the workflow after that.

@srid
Copy link
Member

srid commented May 31, 2024

I just realized that convco which I'm using (as distinct from commitzen which this repo uses) automatically does this -- the linking to commit range on GitHub. Example: https://github.com/juspay/nix-rs/blob/main/CHANGELOG.md (generated by just changelog)

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

Successfully merging this pull request may close these issues.

2 participants