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

Dependencies: pulumi 3.81.0 #2556

Merged
merged 4 commits into from
Sep 12, 2023
Merged

Dependencies: pulumi 3.81.0 #2556

merged 4 commits into from
Sep 12, 2023

Conversation

t0yv0
Copy link
Member

@t0yv0 t0yv0 commented Sep 7, 2023

Proposed changes

The custom script called from custom Make had temporary modifications to setup.py and pyproject.toml.
These are now upstreamed to pulumi/pkg and with version 3.81.0 this happens automatically as part of code generation.
This PR removes temporary script code.

Fixes #2553

Part of the rollout plain in https://github.com/pulumi/home/issues/2883

Related issues (optional)

@t0yv0 t0yv0 requested review from a team and mikhailshilkov September 7, 2023 19:38
@t0yv0 t0yv0 added the impact/no-changelog-required This issue doesn't require a CHANGELOG update label Sep 7, 2023
@github-actions
Copy link

github-actions bot commented Sep 7, 2023

Does the PR have any schema changes?

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

@codecov-commenter
Copy link

codecov-commenter commented Sep 7, 2023

Codecov Report

Merging #2556 (f059312) into master (8b5fad7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2556   +/-   ##
=======================================
  Coverage   21.09%   21.09%           
=======================================
  Files          38       38           
  Lines        8069     8069           
=======================================
  Hits         1702     1702           
  Misses       6274     6274           
  Partials       93       93           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@t0yv0
Copy link
Member Author

t0yv0 commented Sep 11, 2023

@pulumi/providers ready for a quick look. This closes a build P1.

Copy link
Contributor

@guineveresaenger guineveresaenger left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@t0yv0 t0yv0 merged commit ee65f3e into master Sep 12, 2023
17 checks passed
@t0yv0 t0yv0 deleted the t0yv0/pulumi-3.81.0 branch September 12, 2023 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact/no-changelog-required This issue doesn't require a CHANGELOG update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Workflow failure: weekly-pulumi-update
3 participants