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

Extract all contributors into the changelog credits section in the compressed changelog #184

Merged

Conversation

wlhlm
Copy link
Member

@wlhlm wlhlm commented Dec 5, 2024

The previous behavior was to only include "new" contributors in the credits sections which where drawn from the changelog itself from the new contributor sections for each changed mod. However that does not account for people who have contributed to other repositories in our organization and it sweeps returning contributors under the rug.

It's a good idea to make special mention of new contributors, but that needs a more comprehensive approach most likely pulling from the GitHub API to do it across all repositories.

For demonstration, I have included an updated 2.7.0-RC-2 to -RC-3 changelog.

@wlhlm wlhlm changed the title List all contributors in a the changelog credits section List all contributors in the changelog credits section Dec 5, 2024
@wlhlm wlhlm changed the title List all contributors in the changelog credits section Extract all contributors into the changelog credits section in the compressed changelog Dec 6, 2024
@wlhlm wlhlm force-pushed the you-get-a-credit-and-you-get-a-credit-and-you-get-a-credit branch from ecbafdb to 34141e9 Compare December 6, 2024 12:24
@wlhlm wlhlm force-pushed the you-get-a-credit-and-you-get-a-credit-and-you-get-a-credit branch from 34141e9 to 7947b45 Compare December 6, 2024 12:33
@boubou19 boubou19 merged commit 8a4ed25 into master Dec 6, 2024
1 check passed
@boubou19 boubou19 deleted the you-get-a-credit-and-you-get-a-credit-and-you-get-a-credit branch December 6, 2024 17:05
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.

3 participants