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 release contributors to CHANGELOG #171

Merged
merged 2 commits into from
Jun 10, 2024
Merged

Conversation

gasparnagy
Copy link
Contributor

In order to give more credit to the people who contribute to the open-source project, I recommend listing all contributors of a particular release in the change log (with GitHub handle & in alphabetical order).

My goal is to make this as an ongoing thing (everyone should add their own name as part of PR), but of course we can fix the changelog backwards as well.

What do you think?

@Code-Grump
Copy link
Contributor

Sounds like a nice way to add recognition and encourages updates the change-log. Two thumbs up from me.

@gasparnagy gasparnagy requested a review from a team June 10, 2024 09:26
@ajeckmans
Copy link
Contributor

I agree with @Code-Grump

@gasparnagy
Copy link
Contributor Author

@Code-Grump @ajeckmans pls approve it then. I will try to make a release after lunch.

@gasparnagy gasparnagy merged commit 2189799 into main Jun 10, 2024
4 checks passed
@gasparnagy gasparnagy deleted the add-release-contributors branch June 10, 2024 13:18
gasparnagy added a commit that referenced this pull request Jun 12, 2024
* main:
  Reqnroll.Verify: Support for Verify v24 (Verify.Xunit v24.2.0) for .NET 4.7.2+ and .NET 6.0+. (#151, #170)
  use pull_request_template.md from the .github repo
  Fix spelling mistake in CONTRIBUTING.md
  Bump version
  fix config docs
  Add release contributors to CHANGELOG (#171)
  Update CONTRIBUTING with "draft PR" workflow and no force push after review
  fix CHANGELOG
  Fix project template binding (#169)
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