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

build: update Browserslist db #26733

Merged
merged 2 commits into from
Dec 11, 2024
Merged

build: update Browserslist db #26733

merged 2 commits into from
Dec 11, 2024

Conversation

posthog-bot
Copy link
Contributor

Caniuse database has been updated. Review changes, merge this PR and have a 🍺.

@posthog-bot posthog-bot added dependencies Pull requests that update a dependency file automerge Merge this PR automatically when REQUIRED checks pass labels Dec 6, 2024
@rafaeelaudibert rafaeelaudibert removed the automerge Merge this PR automatically when REQUIRED checks pass label Dec 6, 2024
@rafaeelaudibert
Copy link
Member

Oh what the hell, why did it update so many dependencies 🤦🏻

@rafaeelaudibert
Copy link
Member

Copy link
Contributor

github-actions bot commented Dec 6, 2024

Size Change: 0 B

Total Size: 1.11 MB

ℹ️ View Unchanged
Filename Size
frontend/dist/toolbar.js 1.11 MB

compressed-size-action

@rafaeelaudibert
Copy link
Member

Ok, it updated so many dependencies because we didn't explicitly install caniuse-lite. This should be good to go now.

@rafaeelaudibert rafaeelaudibert self-requested a review December 6, 2024 17:18
@rafaeelaudibert rafaeelaudibert added the automerge Merge this PR automatically when REQUIRED checks pass label Dec 6, 2024
This will allow future `browserlist` update to update this properly. Because we didn't have this before, our CI would run `pnpm up caniuse-lite` and `pnpm` would silently fail and simply run `pnpm up` upgrading everything, which doesn't make sense
@github-actions github-actions bot force-pushed the browserslist-update branch from aeccd37 to 02bb2a1 Compare December 9, 2024 12:11
@posthog-bot posthog-bot merged commit 8541547 into master Dec 11, 2024
96 checks passed
@posthog-bot posthog-bot deleted the browserslist-update branch December 11, 2024 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this PR automatically when REQUIRED checks pass dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants