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

Allow passing --use-local-cdk to prerelease publish #31716

Merged
merged 7 commits into from
Oct 24, 2023

Conversation

erohmensing
Copy link
Contributor

@erohmensing erohmensing commented Oct 23, 2023

What

  • Enable using the local CDK in prerelease publishing. This is useful if a change to a connector that we want to validate requires connected changes to the CDK (which we want to validate before publishing).
  • I couldn't think of a use where we would want to support using the local CDK for actual publishing, so decided to explicitly not allow this.
  • Users can make use of this behavior by adding --use-local-cdk after their --name= argument in the workflow (options to pass to the connectors group).

How

  • Pass connector --use-local-cdk arg into PublishConnectorContext, which will pass it into the ConnectorContext used for build

🚨 User Impact 🚨

None

@vercel
Copy link

vercel bot commented Oct 23, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 24, 2023 4:47pm

@erohmensing erohmensing changed the title pass cdk into publish command Allow passing --use-local-cdk to prerelease publish Oct 23, 2023
@erohmensing erohmensing marked this pull request as ready for review October 23, 2023 16:11
@erohmensing erohmensing requested a review from a team October 23, 2023 16:11
Comment on lines 54 to 55
if use_local_cdk and not self.pre_release:
raise ValueError("Publishing with the local CDK is only supported for pre-release publishing.")
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure if there is a better place to perform this validation? In the publish pipeline perhaps?

@erohmensing erohmensing enabled auto-merge (squash) October 23, 2023 16:42
@erohmensing erohmensing merged commit 1336b4e into master Oct 24, 2023
20 checks passed
@erohmensing erohmensing deleted the ella/publish-local-cdk branch October 24, 2023 17:00
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