Skip to content

ci: remove PAT from release workflow#1497

Merged
michaelfaith merged 1 commit intomainfrom
ci/remove-pat-from-release
Jan 4, 2026
Merged

ci: remove PAT from release workflow#1497
michaelfaith merged 1 commit intomainfrom
ci/remove-pat-from-release

Conversation

@michaelfaith
Copy link
Owner

PR Checklist

Overview

Taking inspiration from yargs, I've updated the workflows so that the release workflow no longer needs a PAT. This means CI won't automatically trigger on Release PRs anymore, but a maintainer can trigger CI by adding a ci label to the PR

Taking inspiration from `yargs`, I've updated the workflows so that the `release` workflow no longer needs a PAT.  This means CI won't automatically trigger on Release PRs anymore, but a maintainer can trigger CI by adding a `ci` label to the PR
@github-actions
Copy link
Contributor

github-actions bot commented Jan 3, 2026

👋 Hi @michaelfaith, thanks for the pull request! A scan flagged some concerns with it. Could you please take a look?

[pr-task-completion] This PR's body is missing [x] checks on the following tasks from the PR template.

Repositories often provide a set of tasks that pull request authors are expected to complete. Those tasks should be marked as completed with a [x] in the pull request description. Please complete those tasks and mark the checks as [x] completed.

[pr-linked-issue] This pull request is not linked as closing any issues. This repository keeps to GitHub issues for discussing potential changes. Most or all changes should be marked as approved in an issue before a pull request is sent to resolve them. To resolve this report:

  • If this is a straightforward documentation change that doesn't need an issue, you can ignore this report
  • If there is a backing issue, add a 'fixes #...' link to the pull request body
  • Otherwise, file an issue explaining what you'd like to happen

🗺️ This message was posted automatically by OctoGuide: a bot for GitHub repository best practices.

@michaelfaith michaelfaith marked this pull request as ready for review January 3, 2026 23:17
@codecov
Copy link

codecov bot commented Jan 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.39%. Comparing base (4d802c5) to head (795b64a).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1497   +/-   ##
=======================================
  Coverage   97.39%   97.39%           
=======================================
  Files          28       28           
  Lines         575      575           
  Branches      169      169           
=======================================
  Hits          560      560           
  Misses          1        1           
  Partials       14       14           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Collaborator

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

Sweet.

@michaelfaith michaelfaith merged commit bc9aa17 into main Jan 4, 2026
15 of 16 checks passed
@michaelfaith michaelfaith deleted the ci/remove-pat-from-release branch January 4, 2026 13:57
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