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

feat!: update imports for K8s 1.27 #64

Merged
merged 5 commits into from
Sep 4, 2023
Merged

Conversation

mrliptontea
Copy link
Member

  • Update packages.
    • Defining types for custom matchers in vitest has changed, so reflected those changes in test/test.d.ts.
    • typeRoots, include, and exclude were incorrect in tsconfig, so removed them, as default values seem to work better.
  • Update NodeJS version.
  • Update imports to Kubernetes 1.27.
    • cdk8s import no longer generates enums from schema - I can't find any related issues but anyway I decided to include our own enums for the string values but not actually forcing them in types, because 1) underlying objects use string, and 2) there may be nested properties, for example initContainers, that use imported types directly, so we can't control whether they require enums or not.
  • Re-format everything with prettier.

@mrliptontea mrliptontea changed the title chore: update dependencies feat!: update imports for K8s 1.27 Sep 4, 2023
@mrliptontea mrliptontea requested a review from a team September 4, 2023 13:11
@mrliptontea mrliptontea merged commit 0e3037d into main Sep 4, 2023
2 checks passed
@mrliptontea mrliptontea deleted the chore_update_dependencies branch September 4, 2023 13:14
@talisaspire
Copy link
Collaborator

🎉 This PR is included in version 4.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@talisaspire
Copy link
Collaborator

🎉 This PR is included in version 4.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants