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

Add ci trigger for sdk-features #961

Merged
merged 2 commits into from
Nov 22, 2022

Conversation

Sushisource
Copy link
Member

Draft until temporalio/features#162 is merged

@Sushisource
Copy link
Member Author

This is working as expected, but breakage from #959 must be addressed in SDK-features

@Sushisource Sushisource force-pushed the sdk-features-ci branch 4 times, most recently from d6ff92e to 66a8e45 Compare November 17, 2022 20:02
@Sushisource
Copy link
Member Author

Oops, forgot to mark this one ready. The breaking change issue still stands, but we don't necessarily need to address it before merge (I also forgot to add the branch override for the go workflow, which I'll be doing soon as part of my next PR to sdk-features, so we could wait until then too).

@Sushisource Sushisource marked this pull request as ready for review November 18, 2022 23:51
@Sushisource Sushisource requested a review from a team as a code owner November 18, 2022 23:51
Copy link
Member

@cretz cretz left a comment

Choose a reason for hiding this comment

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

LGTM. We might as well get CI working even if not in this PR. It'd be a good time to test out our long-lived sdk-features temp branches for breaking changes.

@Sushisource
Copy link
Member Author

I will merge this, and we can figure out the failure-fix-in-branch deal later, since this isn't set to block PRs yet.

@Sushisource Sushisource merged commit 0635622 into temporalio:master Nov 22, 2022
@Sushisource Sushisource deleted the sdk-features-ci branch November 22, 2022 19:50
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.

3 participants