Skip to content

Modernize GitHub release contributors layout#168

Open
nirooxx wants to merge 25 commits into
mainfrom
improve/release-contributors-layout
Open

Modernize GitHub release contributors layout#168
nirooxx wants to merge 25 commits into
mainfrom
improve/release-contributors-layout

Conversation

@nirooxx
Copy link
Copy Markdown
Member

@nirooxx nirooxx commented May 15, 2026

This PR modernizes the GitHub release contributor output.

The release notes now use GitHub-native @mentions in the commit list instead of rendering contributors with a custom HTML table. This makes the release page look cleaner, more consistent, and closer to GitHub’s native release notes style.

nirooxx and others added 21 commits March 27, 2025 09:13
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@nirooxx nirooxx requested review from Copilot and pavelkornev May 15, 2026 06:42
@nirooxx
Copy link
Copy Markdown
Member Author

nirooxx commented May 15, 2026

@copilot resolve the merge conflicts in this pull request

Co-authored-by: nirooxx <51267793+nirooxx@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the release/changelog generation pipeline to produce GitHub-native release notes by embedding @mentions directly in the commit list, removing the previous custom contributor HTML layout. It also refactors the release creation code and tightens several workflow scripts used by the Docker-based GitHub Action.

Changes:

  • Generate commit lists with GitHub-native @mentions (and adjust changelog content accordingly), removing the contributors table output.
  • Refactor createRelease to require env vars explicitly and improve error handling; update Jest tests/mocks accordingly.
  • Improve git ref syncing and PR creation error handling; update Jest config, TS config, and dependencies.

Reviewed changes

Copilot reviewed 10 out of 12 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
tsconfig.json Expands TS include set to cover Jest config (but currently has a duplicate entry).
src/release.ts Refactors env handling and errors; keeps CLI behavior via a top-level catch/exit.
src/__tests__/release.test.ts Rewrites tests to use a dedicated @actions/github mock module and validate output writing.
src/__tests__/mocks/actions-github.ts Adds a lightweight in-repo mock for @actions/github used by Jest.
scripts/collect-commits.sh Replaces contributor HTML generation with commit lines that include @mentions.
scripts/update-changelog.sh Updates changelog content generation to match the new “What’s Changed” commit-only format.
scripts/setup-release.sh Refreshes branches/tags via prune fetches to avoid stale refs during releases.
scripts/entrypoint.sh Masks GITHUB_TOKEN in logs and changes how token presence is reported.
scripts/create-pr.sh Improves PR creation error handling by checking HTTP status and response content.
jest.config.mjs Adds module mapping for ESM + @actions/github mock; excludes tests from coverage collection.
.gitattributes Normalizes line endings (LF) for shell scripts and enables text auto-detection.
package-lock.json Updates dependency lockfile to reflect dependency graph changes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread scripts/entrypoint.sh
Comment thread scripts/create-pr.sh
Comment thread scripts/update-changelog.sh
Comment thread scripts/collect-commits.sh
Comment thread src/__tests__/release.test.ts Outdated
Copy link
Copy Markdown

Copilot AI commented May 15, 2026

@copilot resolve the merge conflicts in this pull request

Merge conflicts resolved in commit 6a93429. The conflicts were in scripts/collect-commits.sh (kept the modernized @mentions implementation) and package-lock.json (accepted updated dependency versions from main). All tests pass.

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