-
Notifications
You must be signed in to change notification settings - Fork 378
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
feat!: GitHub API access via GitHub Apps in workflows #1709
Conversation
Signed-off-by: github-actions <github-actions@github.com>
Codecov Report
@@ Coverage Diff @@
## main #1709 +/- ##
==========================================
+ Coverage 88.06% 89.50% +1.44%
==========================================
Files 132 152 +20
Lines 5109 6177 +1068
Branches 1207 1575 +368
==========================================
+ Hits 4499 5529 +1030
- Misses 610 646 +36
- Partials 0 2 +2
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Like it better like that :) GG
Fixes #1657 Allows projen workflows requiring GitHub API access beyond the permissions of GITHUB_TOKEN to specify access from either a personal access token or a GitHub App. It can be configured like so: ```ts const { javascript, github } = require('projen'); const project = new javascript.NodeProject({ // ...other options githubOptions: { projenCredentials: github.GithubCredentials.fromApp({ ... }), }, }); ``` See https://github.com/projen/projen/blob/rybickic/github-app-access/docs/github.md for more details. This change also removes `projenUpgradeSecret` from NodeProjectOptions, which resulted in a cascade of other breaking changes. Since #1383 (v0.47.0) we've required provide a PROJEN_GITHUB_TOKEN for all projects, but NodeProject still assumed projects were using GITHUB_TOKEN by default, so projects were being created with separate workflows for upgrading projen and for upgrading other dependencies. I've removed that default, though it's still possible to create these kinds of workflows separately (details here: https://github.com/projen/projen/blob/rybickic/github-app-access/docs/node.md). BREAKING CHANGE: The deprecated `projenUpgradeSecret` is removed, and GitHub API access is now customizable through `githubOptions.projenCredentials`. * Separate tasks/workflows for upgrading projen (and approving those PRs) are no longer generated by default. `autoApproveProjenUpgrades`, `projenUpgradeAutoMerge`, and `projenUpgradeSchedule`have been correspondingly removed. --- By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Fixes #1657
Allows projen workflows requiring GitHub API access beyond the permissions of GITHUB_TOKEN to specify access from either a personal access token or a GitHub App. It can be configured like so:
See https://github.com/projen/projen/blob/rybickic/github-app-access/docs/github.md for more details.
This change also removes
projenUpgradeSecret
from NodeProjectOptions, which resulted in a cascade of other breaking changes. Since #1383 (v0.47.0) we've required provide a PROJEN_GITHUB_TOKEN for all projects, but NodeProject still assumed projects were using GITHUB_TOKEN by default, so projects were being created with separate workflows for upgrading projen and for upgrading other dependencies. I've removed that default, though it's still possible to create these kinds of workflows separately (details here: https://github.com/projen/projen/blob/rybickic/github-app-access/docs/node.md).BREAKING CHANGE: The deprecated
projenUpgradeSecret
is removed, and GitHub API access is now customizable throughgithubOptions.projenCredentials
.autoApproveProjenUpgrades
,projenUpgradeAutoMerge
, andprojenUpgradeSchedule
have been correspondingly removed.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.