-
Notifications
You must be signed in to change notification settings - Fork 3
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
chore: use cdklabs-projen-project-types #285
chore: use cdklabs-projen-project-types #285
Conversation
We are standardizing repository setups to improve maintainability. If you have any questions regarding this change, please reach out to us.
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.
In addition we need to either 1) click a button in the settings to allow auto merge or 2) set up the automation in cdk-ops to do this for us. @comcalvi may have already started work on a projen task that can configure repository settings.
I have started clicking these buttons manually. Also need to enable the merge queue via ops me thinks. |
autoApproveOptions: { | ||
allowedUsernames: ['cdklabs-automation'], | ||
secret: 'GITHUB_TOKEN', | ||
}, | ||
autoApproveUpgrades: true, |
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.
These are also not needed.
We are standardizing repository setups to improve maintainability.
If you have any questions regarding this change, please reach out to us.