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

Bump and test content-api-client preview release version #312

Merged
merged 1 commit into from
Apr 23, 2024

Conversation

Divs-B
Copy link
Contributor

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

We are testing content-entity latest release that is out after adopting gha-scala-release-process init,
the version is updated in content-api-model and content-api-client so we need to test these libraries preview releases in fapi client too.

The intention is to test all these preview releases in any of client consuming these linbraries, we have apple-news and pubflow for example.

How to test

Make a preview release. test in apple-news

How can we measure success?

Application should compile with no version conflicts between libraries
Application shoud run well without any erros.

@gu-scala-library-release
Copy link
Contributor

@Divs-B has published a preview version of this PR with release workflow run #18, based on commit 53de8b7:

6.0.0-PREVIEW.dbtest-content-api-client-prew-version-bump.2024-03-27T1128.53de8b71

Want to make another preview release?

Click 'Run workflow' in the GitHub UI, specifying the db/test-content-api-client-prew-version-bump branch, or use the GitHub CLI command:

gh workflow run release.yml --ref db/test-content-api-client-prew-version-bump

Want to make a full release after this PR is merged?

Click 'Run workflow' in the GitHub UI, leaving the branch as the default, or use the GitHub CLI command:

gh workflow run release.yml

@Divs-B Divs-B force-pushed the db/test-content-api-client-prew-version-bump branch from 057c1cb to 57168cb Compare April 23, 2024 10:06
@Divs-B
Copy link
Contributor Author

Divs-B commented Apr 23, 2024

Bumped production release and tested locally via sbt test . tests are passing.

@Divs-B
Copy link
Contributor Author

Divs-B commented Apr 23, 2024

Thanks @davidfurey and @rtyley for approving. I will merge and make prod release.
Then we will be all done to use all in apple-news or pubflow or any client using them.

@Divs-B Divs-B merged commit 431f2b4 into main Apr 23, 2024
1 check passed
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