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

Adopt GHA Scala Library Release Workflow #1769

Merged
merged 1 commit into from Apr 1, 2024

Conversation

rtyley
Copy link
Contributor

@rtyley rtyley commented Apr 1, 2024

As mentioned in #1763, we're switching to use gha-scala-library-release-workflow for releasing the Scanamo to Maven Central. This is a change from the release process based on sbt-ci-release that was introduced with #280 - it used to be that to trigger a release, we would push a git tag with a manually-selected version number (eg v1.0.3), the new process is now to run the Release workflow manually, which will automatically determine a version number based on the version-compatibility checks performed by sbt-version-policy.

In order to do this, we've had to follow the one-off organisation-setup steps in:

https://github.com/guardian/gha-scala-library-release-workflow/blob/main/docs/org-setup.md

...which included creating a GitHub App for the Scanamo organisation:

https://github.com/apps/scala-library-release-scanamo

Additionally, this PR contains the per-repo modifications required by:

https://github.com/guardian/gha-scala-library-release-workflow/blob/main/docs/configuration.md

Remaining steps

As mentioned in #1763, we're
switching to use `gha-scala-library-release-workflow` for releasing the
Scanamo to Maven Central. This is a change from the release process based
on `sbt-ci-release` that was introduced with #280 -
it used to be that to trigger a release, we would push a git tag with a manually-selected
version number (eg `v1.0.3`), the new process is now to run the `Release` workflow manually,
which will automatically determine a version number based on the binary-compatibility checks
performed by `sbt-version-policy`.

In order to do this, we've had to follow the one-off organisation-setup steps in:

https://github.com/guardian/gha-scala-library-release-workflow/blob/main/docs/org-setup.md

...which included creating a GitHub App for the Scanamo organisation:

https://github.com/apps/scala-library-release-scanamo

Additionally, this PR contains the per-repo modifications required by:

https://github.com/guardian/gha-scala-library-release-workflow/blob/main/docs/configuration.md
publishingSettings,
noPublishSettings,
publish / skip := true,
// releaseVersion := ReleaseVersion.fromAggregatedAssessedCompatibilityWithLatestRelease().value,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This had to be disabled due to v1.0.2 failing and guardian/gha-scala-library-release-workflow#33

@rtyley rtyley merged commit d507c90 into main Apr 1, 2024
8 checks passed
@rtyley rtyley deleted the adopt-GHA-Scala-Library-Release-Workflow branch April 1, 2024 11:45
rtyley added a commit that referenced this pull request Apr 1, 2024
Following on from #1769,
and as mentioned in:

https://github.com/guardian/gha-scala-library-release-workflow/blob/main/docs/configuration.md#recommended-sbt-settings

...unfortunately it's difficult to figure out how to modify
gha-scala-library-release-workflow to start the DynamodDB Local instance
to get tests to pass. So, either we modify Scanamo's sbt build to
automatically start the DynamodDB Local instance when the tests start -
or we just skip tests during release right now.
rtyley added a commit that referenced this pull request Apr 1, 2024
As mentioned in #1769 (review),
when we adopted `gha-scala-library-release-workflow` in #1769
we couldn't immediately enable syntactic-compatibility-checking as the previous release
(v1.0.2) had failed.
@rtyley
Copy link
Contributor Author

rtyley commented Apr 1, 2024

After ce12e44 & d777c21, releases are working well 👍

image

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

1 participant