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

Remove some unused versioning code #2441

Merged

Conversation

thomas11
Copy link
Contributor

@thomas11 thomas11 commented May 9, 2023

Noticed by chance, probably a follow-up to #2428

thomas11 and others added 7 commits April 18, 2023 22:04
This change should be enough to enable CI acceptance test runs for PRs
into the new v2.0-pre branch.

As far as I can see, support for releasing pre-releases is already there
in release.yaml.
- Upgrade schema-tools to 0.2.2 for squeeze improvement
- First iteration of squeezing the v2 spec
- Rename files and code for clarity, remove some unused code
- Move squeezing to just before schema generation
- Preserve resources in v1-lock when squeezing
- Groundwork for generating the v2 schema, but not plumbed all the way
through yet.

---------

Co-authored-by: Daniel Bradley <daniel@pulumi.com>
Cherry-pick of b68d1c5 from #2410 to get `master` and `v2.0-pre` in
sync

---------

Co-authored-by: Daniel Bradley <daniel@pulumi.com>
@github-actions
Copy link

github-actions bot commented May 9, 2023

Does the PR have any schema changes?

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

Copy link
Member

@danielrbradley danielrbradley 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, but think this can go straight into master instead of v2.0-pre.

@thomas11 thomas11 changed the base branch from v2.0-pre to master May 9, 2023 19:16
@thomas11 thomas11 enabled auto-merge (squash) May 9, 2023 19:17
@thomas11 thomas11 merged commit 747ccc7 into master May 9, 2023
13 checks passed
@thomas11 thomas11 deleted the tkappler/unused-ReadAndApplyProvidersTransformations branch May 9, 2023 19:51
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.

None yet

2 participants