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

🦋 Release package updates #1731

Merged
merged 1 commit into from
Jan 21, 2025
Merged

🦋 Release package updates #1731

merged 1 commit into from
Jan 21, 2025

Conversation

gu-changesets-release-pr[bot]
Copy link
Contributor

@gu-changesets-release-pr gu-changesets-release-pr bot commented Jan 9, 2025

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.

Releases

@guardian/[email protected]

Major Changes

Minor Changes

  • 627e730: Delete article-aside-adverts as it's no longer used

Patch Changes

  • 3285a0b: Remove amIUsed calls
  • fda5154: removes sonobi ssr from prebid configuaration

@gu-changesets-release-pr gu-changesets-release-pr bot requested a review from a team as a code owner January 9, 2025 15:09
Copy link
Contributor

github-actions bot commented Jan 9, 2025

Ad load time test results

For consented, top-above-nav took on average 4188ms to load.
For consentless, top-above-nav took on average 2769ms to load.

Test conditions:

  • 5mbps download speed
  • 1.5mbps upload speed
  • 150ms latency

@gu-changesets-release-pr gu-changesets-release-pr bot force-pushed the changeset-release/main branch 11 times, most recently from 9af6e61 to 8e09170 Compare January 16, 2025 11:27
@gu-changesets-release-pr gu-changesets-release-pr bot force-pushed the changeset-release/main branch 2 times, most recently from cf1252e to 65a6ffc Compare January 17, 2025 11:03
Copy link
Contributor

@dskamiotis dskamiotis left a comment

Choose a reason for hiding this comment

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

I assume this is the process which we are continuing to deliver (along side the riffraff auto release?) via npmjs for commercial as a import lib fro DCR? @Jakeii

@arelra
Copy link
Member

arelra commented Jan 21, 2025

I assume this is the process which we are continuing to deliver (along side the riffraff auto release?) via npmjs for commercial as a import lib fro DCR? @Jakeii

Yes we still need version bumps to library functions required for Frontend and DCR.

@arelra arelra merged commit 09bc5d7 into main Jan 21, 2025
14 checks passed
@arelra arelra deleted the changeset-release/main branch January 21, 2025 10:23
@prout-bot
Copy link
Collaborator

Seen on PROD (created by @gu-changesets-release-pr[bot] and merged by @arelra 2 minutes and 7 seconds ago) Please check your changes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants