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

Prepare for v1.5.2 release #149

Merged
merged 4 commits into from
Sep 16, 2024
Merged

Prepare for v1.5.2 release #149

merged 4 commits into from
Sep 16, 2024

Conversation

rquitales
Copy link
Member

@rquitales rquitales commented Sep 16, 2024

This PR contains 3 changes/commits and is intended to be rebased and merged to maintain commit history.

Changes made:

  1. Removed go mod replacements with upstream changes now merged to master.
  2. Fixed generating Golang types for StringMapArrayMap properties. We now correctly type this nested field. An additional CRD is now installed and compiled in our integration test suite.
  3. Fixed setting v4.18.0 as the p-k dependency for generated Python packages.

Related Issues:

Resolves: #147, resolves #148

We were previously not nesting maps of arrays, and treating them as
generic types instead. This is an issue in Golang codegen, generating
generic Pulumi input/output types that don't exists. This commit ensures
we also flatten map of arrays so Golang codegen works.

Fixes: #147
@rquitales rquitales requested a review from blampe September 16, 2024 23:25
@rquitales rquitales self-assigned this Sep 16, 2024
pkg/codegen/packagegenerator.go Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@rquitales rquitales enabled auto-merge (rebase) September 16, 2024 23:38
Copy link

codecov bot commented Sep 16, 2024

Codecov Report

Attention: Patch coverage is 80.00000% with 7 lines in your changes missing coverage. Please review.

Project coverage is 78.34%. Comparing base (f670a71) to head (92dc921).
Report is 4 commits behind head on master.

Files with missing lines Patch % Lines
pkg/codegen/customresourcegenerator.go 76.66% 4 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
+ Coverage   78.10%   78.34%   +0.23%     
==========================================
  Files          17       17              
  Lines         708      725      +17     
==========================================
+ Hits          553      568      +15     
- Misses         87       88       +1     
- Partials       68       69       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rquitales rquitales merged commit d05a577 into master Sep 16, 2024
5 checks passed
@rquitales rquitales deleted the rquitales/update-p-k branch September 16, 2024 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants