Skip to content

chore: update release drafter workflow to include autolabeler and adjust permissions#56

Merged
wax911 merged 2 commits intomainfrom
ci/workflow-updates-to-release-drafter
Apr 26, 2026
Merged

chore: update release drafter workflow to include autolabeler and adjust permissions#56
wax911 merged 2 commits intomainfrom
ci/workflow-updates-to-release-drafter

Conversation

@wax911
Copy link
Copy Markdown
Member

@wax911 wax911 commented Apr 26, 2026

AniTrend Pull Request

Thank you for contributing! Please take a moment to review our contributing guidelines to make the process easy and effective for everyone involved.

Please open an issue before embarking on any significant pull request, especially those that add a new library or change existing tests, otherwise you risk spending a lot of time working on something that might not end up being merged into the project.

Before opening a pull request, please ensure you've done the following:

  • You have followed our contributing guidelines
  • Double checked that your branch is based on main and targets main (where applicable)
  • Pull request has tests (if applicable)
  • Documentation is updated (if necessary)
  • Description explains the issue/use-case resolved

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Enhancement (Improves existing functionality)

IMPORTANT: By submitting a patch, you agree to allow the project owners to license your work under the terms of the Apache License, Version 2.0.

…ust permissions

Co-authored-by: Copilot <copilot@github.com>
Copilot AI review requested due to automatic review settings April 26, 2026 10:20
@anitrend anitrend Bot added the 👷 ci label Apr 26, 2026
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

Updates the GitHub Actions release automation by separating PR autolabeling into its own workflow and tightening permissions for the release-drafter workflow.

Changes:

  • Remove pull_request trigger from the main release-drafter workflow and reduce its PR permission scope.
  • Add a dedicated release-drafter autolabeler workflow that runs on PR events (including fork-safe handling via pull_request_target).
  • Adjust release-drafter inputs (e.g., commitish) and reformat some long YAML values.

Reviewed changes

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

File Description
.github/workflows/release-drafter.yml Restricts triggers/permissions and tweaks release-drafter + version bump PR creation configuration.
.github/workflows/release-drafter-autolabeler.yml Introduces a new workflow to automatically label PRs using Release Drafter’s autolabeler.

Comment thread .github/workflows/release-drafter.yml Outdated
Comment thread .github/workflows/release-drafter-autolabeler.yml
Comment thread .github/workflows/release-drafter.yml
Comment thread .github/workflows/release-drafter.yml
Comment thread .github/workflows/release-drafter.yml Outdated
Copilot AI review requested due to automatic review settings April 26, 2026 11:09
@wax911 wax911 enabled auto-merge (squash) April 26, 2026 11:11
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Auto approved automated PR

@wax911 wax911 merged commit c9f1471 into main Apr 26, 2026
16 checks passed
@wax911 wax911 deleted the ci/workflow-updates-to-release-drafter branch April 26, 2026 11:11
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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Comment on lines +111 to +112
commit-message: "chore: bump version to ${{
needs.update_release_draft.outputs.resolved_version }}"
Copy link

Copilot AI Apr 26, 2026

Choose a reason for hiding this comment

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

commit-message is split across multiple YAML lines inside a double-quoted string. YAML will include a newline here, so the actual commit subject becomes chore: bump version to ${{ and the resolved_version ends up on the next line, producing an unintended (and likely invalid) commit message. Keep the expression on a single line or use a folded scalar (>-) so the resulting string has no newline.

Suggested change
commit-message: "chore: bump version to ${{
needs.update_release_draft.outputs.resolved_version }}"
commit-message: "chore: bump version to ${{ needs.update_release_draft.outputs.resolved_version }}"

Copilot uses AI. Check for mistakes.
Comment on lines +4 to +8
pull_request:
types: [ opened, reopened, synchronize ]
pull_request_target:
types: [ opened, reopened, synchronize ]

Copy link

Copilot AI Apr 26, 2026

Choose a reason for hiding this comment

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

This workflow listens to both pull_request and pull_request_target. For same-repo PRs, both events will fire, so you’ll get an extra (skipped) workflow run due to the if: guard. To reduce runner usage/noise, consider using a single event (typically pull_request_target) and handling fork vs same-repo behavior solely in the if: condition.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants