Skip to content

Conversation

@darkynt
Copy link

@darkynt darkynt commented Dec 22, 2025

  • Adds initial tracing and logging default middleware reference implementations
  • Adds some quick precommit hook and local environment fixes

@darkynt darkynt force-pushed the feat/udp-48-post-cdk-tracing branch from fb993b2 to 71b906e Compare December 22, 2025 04:42
@darkynt darkynt force-pushed the feat/udp-48-post-cdk-tracing branch from 71b906e to ecfa6cc Compare December 22, 2025 04:42
hooks:
- id: detect-secrets
args: ['--baseline', '.secrets.baseline']
exclude: pnpm-lock.yaml
Copy link
Author

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",
Copy link
Author

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(

@darkynt darkynt marked this pull request as ready for review December 23, 2025 14:02
@darkynt darkynt changed the title [Draft] Feat/udp 48 post cdk tracing Feat/udp 48 post cdk tracing Dec 23, 2025
@darkynt darkynt force-pushed the feat/udp-48-post-cdk-tracing branch from 6a817c5 to fc47671 Compare December 23, 2025 14:11
@darkynt darkynt force-pushed the feat/udp-48-post-cdk-tracing branch from fc47671 to 5808dce Compare December 23, 2025 14:26
cdk/project.json Outdated
"executor": "nx:run-commands",
"options": {
"command": "npx cdk bootstrap",
"command": "DEVELOPER_ID=$(./scripts/set-developer.sh --value) npx cdk bootstrap",
Copy link
Contributor

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
},
Copy link
Contributor

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

Copy link
Author

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

Copy link
Author

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.

@darkynt darkynt changed the title Feat/udp 48 post cdk tracing DontMerge - superseded UDP-48 Dec 23, 2025
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.

3 participants