-
Notifications
You must be signed in to change notification settings - Fork 1
DontMerge - superseded UDP-48 #36
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
base: main
Are you sure you want to change the base?
Conversation
fb993b2 to
71b906e
Compare
71b906e to
ecfa6cc
Compare
| hooks: | ||
| - id: detect-secrets | ||
| args: ['--baseline', '.secrets.baseline'] | ||
| exclude: pnpm-lock.yaml |
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.
not a fan of this but the interface seems to be a string instead of an array/list
| "build:affected": "npx nx affected --target=build", | ||
| "lint:all": "npx nx run-many --target=lint", | ||
| "lint:affected": "npx nx affected --target=lint", | ||
| "lint:affected": "npx nx affected --target=lint --outputStyle static", |
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.
quick fix for hanging precommit hooks backgrounded process(
6a817c5 to
fc47671
Compare
fc47671 to
5808dce
Compare
cdk/project.json
Outdated
| "executor": "nx:run-commands", | ||
| "options": { | ||
| "command": "npx cdk bootstrap", | ||
| "command": "DEVELOPER_ID=$(./scripts/set-developer.sh --value) npx cdk bootstrap", |
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.
We can revert this so all cdk is bootstapped to the same s3
project.json
Outdated
| "cwd": "." | ||
| }, | ||
| "cache": 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.
this has been removed as it was TF related
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.
Good spot, no idea what merge-hell kept it here O_o
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.
ahh. NX magic tooling did it.
Uh oh!
There was an error while loading. Please reload this page.