Skip to content

Conversation

Fweddi
Copy link
Contributor

@Fweddi Fweddi commented Oct 1, 2025

What does this change?

sbt-scrooge-typescript had TS compilation issues (caused by an old version of Typescript used in the test packages - guardian/scrooge-extras#42) and Thrift serialisation issues (resolved by downgrading Thrift - guardian/scrooge-extras#39).

This meant that we were unable to publish apps-rendering-api-models to NPM:

Version 4.0.0 of sbt-scrooge-typescript (guardian/scrooge-extras#39) should resolve these issues.

Testing

✅ Upgrading to 4.0.0 should unblock the release workflow: https://github.com/guardian/apps-rendering-api-models/actions/runs/18189690737
✅ And unblock downstream consumption by DCR: guardian/dotcom-rendering#14533

@Fweddi Fweddi marked this pull request as ready for review October 1, 2025 14:05
@Fweddi Fweddi requested review from a team as code owners October 1, 2025 14:05
@Fweddi Fweddi force-pushed the fp/bump-scrooge-ts branch from abe0bef to 19e74e4 Compare October 1, 2025 14:07
gu-scala-library-release bot added a commit that referenced this pull request Oct 1, 2025
…by Fweddi

Fweddi published release version 13.0.1-PREVIEW.fpbump-scrooge-ts.2025-10-01T1413.19e74e41
using gha-scala-library-release-workflow: https://github.com/guardian/gha-scala-library-release-workflow

Release-Version: 13.0.1-PREVIEW.fpbump-scrooge-ts.2025-10-01T1413.19e74e41
Release-Initiated-By: https://github.com/Fweddi
Release-Workflow-Run: https://github.com/guardian/apps-rendering-api-models/actions/runs/18165055968
Release-Notes: #106
@gu-scala-library-release
Copy link
Contributor

@Fweddi has published a preview version of this PR with release workflow run #20, based on commit 19e74e4:

13.0.1-PREVIEW.fpbump-scrooge-ts.2025-10-01T1413.19e74e41

Want to make another preview release?

Click 'Run workflow' in the GitHub UI, specifying the fp/bump-scrooge-ts branch, or use the GitHub CLI command:

gh workflow run release.yml --ref fp/bump-scrooge-ts

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

gu-scala-library-release bot added a commit that referenced this pull request Oct 2, 2025
…by Fweddi

Fweddi published release version 13.0.1-PREVIEW.fpbump-scrooge-ts.2025-10-02T0957.15fda4f0
using gha-scala-library-release-workflow: https://github.com/guardian/gha-scala-library-release-workflow

Release-Version: 13.0.1-PREVIEW.fpbump-scrooge-ts.2025-10-02T0957.15fda4f0
Release-Initiated-By: https://github.com/Fweddi
Release-Workflow-Run: https://github.com/guardian/apps-rendering-api-models/actions/runs/18189690737
Release-Notes: #106
@gu-scala-library-release
Copy link
Contributor

@Fweddi has published a preview version of this PR with release workflow run #21, based on commit 15fda4f:

13.0.1-PREVIEW.fpbump-scrooge-ts.2025-10-02T0957.15fda4f0

Want to make another preview release?

Click 'Run workflow' in the GitHub UI, specifying the fp/bump-scrooge-ts branch, or use the GitHub CLI command:

gh workflow run release.yml --ref fp/bump-scrooge-ts

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

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.

Looks good! Obviously, just need to update to a non-preview version first.

addSbtPlugin("com.twitter" % "scrooge-sbt-plugin" % "22.1.0")

addSbtPlugin("com.gu" % "sbt-scrooge-typescript" % "2.0.0")
addSbtPlugin("com.gu" % "sbt-scrooge-typescript" % "4.0.0-PREVIEW.fpdowngrade-scrooge.2025-10-02T0939.1433794d")
Copy link
Member

Choose a reason for hiding this comment

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

Just need to update this once guardian/scrooge-extras#39 is merged and released, so that we can use the non-preview version here!

@Fweddi Fweddi merged commit 9328b16 into main Oct 6, 2025
3 checks passed
@Fweddi Fweddi deleted the fp/bump-scrooge-ts branch October 6, 2025 10:18
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