Skip to content

Clean up project#5250

Open
pmairoldi wants to merge 18 commits intomasterfrom
pmairoldi/ci
Open

Clean up project#5250
pmairoldi wants to merge 18 commits intomasterfrom
pmairoldi/ci

Conversation

@pmairoldi
Copy link
Copy Markdown
Collaborator

@pmairoldi pmairoldi commented Mar 5, 2026

Changes:

  • update the GitHub action to pass again
  • add GitHub action to create changelog
  • fix rounded bar charts on macOS
  • update supported macOS version
  • update Xcode project to Xcode 26
  • fix tests

Copilot AI review requested due to automatic review settings March 5, 2026 02:55
Copy link
Copy Markdown

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

Adds a GitHub Actions workflow that generates and commits CHANGELOG.md when a release is published.

Changes:

  • Introduces .github/workflows/changelog.yml to run on release.published
  • Generates CHANGELOG.md via a third-party changelog generator action
  • Commits and pushes the updated changelog back to the repository

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

steps:
- uses: actions/checkout@v4
with:
ref: master
Copy link

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

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

This repository appears to use main (per PR metadata/diff context), but the workflow checks out master. This will fail in repos without a master branch and also risks pushing the changelog to the wrong branch. Consider checking out the default branch (e.g., ref: ${{ github.event.repository.default_branch }}) or explicitly main if that’s the intended target.

Suggested change
ref: master
ref: ${{ github.event.repository.default_branch }}

Copilot uses AI. Check for mistakes.
- name: Generate changelog
uses: janheinrichmerker/action-github-changelog-generator@v2.4
with:
token: ${{ secrets.CHANGELOG_GITHUB_TOKEN }}
Copy link

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

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

Requiring a custom secret token can make this workflow fail in environments where CHANGELOG_GITHUB_TOKEN isn’t configured. If elevated permissions aren’t required, prefer the built-in token (e.g., ${{ github.token }} / ${{ secrets.GITHUB_TOKEN }}) and rely on permissions: contents: write for pushing.

Suggested change
token: ${{ secrets.CHANGELOG_GITHUB_TOKEN }}
token: ${{ github.token }}

Copilot uses AI. Check for mistakes.
fetch-depth: 0

- name: Generate changelog
uses: janheinrichmerker/action-github-changelog-generator@v2.4
Copy link

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

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

For supply-chain hardening, third-party GitHub Actions should be pinned to a full commit SHA rather than a mutable tag (like v2.4). Pinning reduces the risk of unexpected changes if the tag is moved.

Suggested change
uses: janheinrichmerker/action-github-changelog-generator@v2.4
uses: janheinrichmerker/action-github-changelog-generator@0123456789abcdef0123456789abcdef01234567 # v2.4

Copilot uses AI. Check for mistakes.
@pmairoldi pmairoldi changed the title add changelog pipeline Clean up project Mar 7, 2026
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.

2 participants