Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add post-release manual step #5449

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

isaacroldan
Copy link
Contributor

@isaacroldan isaacroldan commented Feb 21, 2025

WHY are these changes introduced?

To improve the post-release workflow by consolidating GitHub authentication logic and adding a new post-release script that automates documentation and homebrew updates.

WHAT is this pull request doing?

  • Extracts common GitHub authentication logic into a new github-utils.js module
  • Creates a new post-release script that ensures releases are made from stable branches
  • Moves command execution logic into a separate run-command.js module
  • Adds a new post-release npm script to package.json
  • Updates existing scripts to use the new shared authentication utilities

How to test your changes?

TBD

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Copy link
Contributor

github-actions bot commented Feb 21, 2025

Coverage report

St.
Category Percentage Covered / Total
🟡 Statements 75.91% 9110/12001
🟡 Branches 71.06% 4454/6268
🟡 Functions 75.52% 2375/3145
🟡 Lines
76.47% (+0.01% 🔼)
8604/11252
Show files with reduced coverage 🔻
St.
File Statements Branches Functions Lines
🟢
... / app-event-watcher.ts
95.18% (-1.2% 🔻)
86.49% (-2.7% 🔻)
95.45% 100%

Test suite run success

2065 tests passing in 922 suites.

Report generated by 🧪jest coverage report action from a5b6894

@isaacroldan isaacroldan force-pushed the 02-21-add_post-release_manual_step branch from e8d55bf to 269851e Compare February 21, 2025 12:27
@isaacroldan isaacroldan marked this pull request as ready for review February 21, 2025 17:46
@isaacroldan isaacroldan requested a review from a team as a code owner February 21, 2025 17:46
@isaacroldan isaacroldan force-pushed the 02-21-add_post-release_manual_step branch from 269851e to 140547a Compare February 21, 2025 17:51
@isaacroldan isaacroldan force-pushed the 02-21-add_post-release_manual_step branch from 140547a to a5b6894 Compare February 21, 2025 17:56
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.

1 participant