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

chore: update deps and logging #284

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Conversation

drduhe
Copy link
Collaborator

@drduhe drduhe commented Feb 5, 2025

Issue #, if available: n/a

Notes

  • Improved logging format for deployment context values.
  • Ensured consistent structured output for readability.
  • Skipped logging of undefined values to reduce noise.

CDK Synth Output

  • Formatted values as key-value lists for better readability.
  • Ensured structured output for both primitive values and objects.
  • Skipped undefined values to keep logs clean and relevant.

package-lock.json

  • Updated dependencies to use latest greatest libraries.

Testing

Checkout the new synth outputs you can see!

 > cdk synth

✅ "Project Name":
  - "OSML"
✅ "Deployment Account ID":
  - id: 409719124294
  - region: us-west-2
  - prodLike: false
✅ "Deploying Model Runner":
  - deploy: true
✅ "Deploying Tile Server":
  - deploy: true
✅ "Deploying Data Intake":
  - deploy: true
✅ "Deploying Data Catalog":
  - deploy: true
✅ "Deploying Test Model Endpoints":
  - deploy: true
✅ "Deploying Test Imagery":
  - deploy: true
⚠️ CDK-Nag Security Checks Disabled.

Before you submit a pull request, please make sure you have to following:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@drduhe drduhe force-pushed the chore/update-deps-and-logging branch 3 times, most recently from bb72b24 to 48a008a Compare February 5, 2025 18:02
@drduhe drduhe force-pushed the chore/update-deps-and-logging branch from 48a008a to 742b2e7 Compare February 6, 2025 19:26
"ts-jest": "^29.2.3",
"typescript": "^5.5.4"
"lint-staged": "^15.4.3",
"osml-cdk-constructs": "file:lib/osml-cdk-constructs",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You will need to undo this and change it to ^2.0.8

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then run: npm update then npm i

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.

2 participants