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

Respect schema version #1539

Merged
merged 3 commits into from
May 16, 2024
Merged

Respect schema version #1539

merged 3 commits into from
May 16, 2024

Conversation

danielrbradley
Copy link
Member

Part of pulumi/ci-mgmt#915

  • Enable respectSchemaVersion for python, go, nodejs and dotnet.
  • Inject version during SDK generation
  • Upgrade pu/pu CLI to ensure we have the extra version argument available.
  • Tidy makefile:
    • Remove manual updating of the version number for python and nodejs now the version number is already set.
    • Fix dotnet so the version.txt has the provider version not the dotnet SDK version.
    • Remove unneeded shell wrapping in bin/pulumi-java-gen target which actually causes the download to happen while make is calculating the target rather than when it runs the target.

- Upgrade pu/pu CLI to ensure we have the extra version argument available.
- Remove manual updating of the version number for python and nodejs now the version number is already set.
- Fix dotnet so the version.txt has the provider version *not* the dotnet SDK version.
- Remove unneeded shell wrapping in bin/pulumi-java-gen target which actually causes the download to happen while make is calculating the target rather than when it runs the target.
@danielrbradley danielrbradley requested review from t0yv0, flostadler, corymhall and a team May 16, 2024 10:48
@danielrbradley danielrbradley self-assigned this May 16, 2024
Copy link
Contributor

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

Copy link

codecov bot commented May 16, 2024

Codecov Report

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

Project coverage is 25.09%. Comparing base (c32eb70) to head (6627f74).

Files Patch % Lines
provider/pkg/schema/gen.go 0.00% 7 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1539      +/-   ##
==========================================
- Coverage   25.12%   25.09%   -0.03%     
==========================================
  Files          34       34              
  Lines        4542     4547       +5     
==========================================
  Hits         1141     1141              
- Misses       3231     3236       +5     
  Partials      170      170              

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

@danielrbradley danielrbradley merged commit 48cc36f into master May 16, 2024
17 checks passed
@danielrbradley danielrbradley deleted the respect-schema-version branch May 16, 2024 12:39
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