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

Temporary disable compatibility check on releases #38

Merged
merged 1 commit into from
Mar 26, 2024

Conversation

Divs-B
Copy link
Contributor

@Divs-B Divs-B commented Mar 26, 2024

Suggested by @rtyley we can disable the releases checks to make succesful release this time, the release process is currently trying to find thrift latest release of 3.0.1 which is not available due to missed thing in PR #32 which got fixed in PR #36.

To fix the releases we will disable release comparison and make the release and then will enable again once releases goes well.

#36 (comment)

How to test

once this gets merged we will make release

How can we measure success?

release should go well

Suggested by Roberto we can disable the releases checks to make succesful release this time, the release process is currently trying to find thrift latest release of 3.0.1 which is not available hence we will disable release comparison and make the release and then will enable again once releases goes well.

#36 (comment)
@Divs-B Divs-B requested a review from rtyley March 26, 2024 13:31
@Divs-B Divs-B requested review from a team as code owners March 26, 2024 13:31
Copy link
Member

@rtyley rtyley left a comment

Choose a reason for hiding this comment

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

@Divs-B Divs-B merged commit 50ca1f6 into main Mar 26, 2024
1 check passed
@Divs-B
Copy link
Contributor Author

Divs-B commented Mar 26, 2024

Release is done with no error this time.

image

Divs-B added a commit that referenced this pull request Mar 26, 2024
As we got successful releases after PR #38 we can now put back AssessedCompatibilityWithLatestReleas in our sbt file.
@rtyley
Copy link
Member

rtyley commented Mar 26, 2024

Release is done with no error this time.

Brilliant! Don't forget to revert this PR now - and then do another release afterwards, which should now succeed!

@Divs-B
Copy link
Contributor Author

Divs-B commented Mar 26, 2024

Release is done with no error this time.

Brilliant! Don't forget to revert this PR now - and then do another release afterwards, which should now succeed!

Ah good spot! Thank you for reminding that we can revert this PR to get back the change we want and no need to extra PR for that.
I will do the revert 👍
Thanks for all help in sorting this out @rtyley

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