Skip to content

fix: credit changelog co-authors - #2528

Merged
andygrove merged 1 commit into
apache:mainfrom
andygrove:andygrove/changelog-script-review
Oct 3, 2026
Merged

andygrove merged 1 commit into
apache:mainfrom
andygrove:andygrove/changelog-script-review

Conversation

@andygrove

Copy link
Copy Markdown
Member

Summary

  • credit every recognized commit author on multi-author pull requests
  • include squashed co-authors in the contributor count and credits table
  • tolerate commits whose author is not linked to a GitHub account
  • retain Ballista-specific dependency filtering and safe subprocess usage

This ports the recent attribution improvement from apache/datafusion-comet#6358 while preserving the improvements already present in Ballista.

Testing

  • uvx --from ruff==0.14.13 ruff check dev/release/generate-changelog.py
  • uvx --from ruff==0.14.13 ruff format --check dev/release/generate-changelog.py
  • mocked co-author and nullable-author attribution tests
  • generated the 54.0.0..54.1.0 changelog against the live GitHub API

@andygrove
andygrove merged commit d6d8bd9 into apache:main Oct 3, 2026
3 checks passed
@andygrove
andygrove deleted the andygrove/changelog-script-review branch October 3, 2026 15:19
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