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

ci: Report WASM size differences on pull requests #4561

Merged
merged 5 commits into from
Dec 13, 2023
Merged

Conversation

SevInf
Copy link
Member

@SevInf SevInf commented Dec 13, 2023

Part of https://github.com/prisma/team-orm/issues/666

Adds CI job that builds base branch and PR, compares their size and
posts a comment. We could not reuse size-limit action we use for client
since it's very JS centric.

@SevInf SevInf added this to the 5.8.0 milestone Dec 13, 2023
Copy link
Contributor

github-actions bot commented Dec 13, 2023

WASM Size

Engine This PR Base branch Diff
WASM 3.2MiB 3.2MiB 0B
WASM (gzip) 1.3MiB 1.3MiB 4B

Part of prisma/team-orm#666

Adds CI job that builds base branch and PR, compares their size and
posts a comment. We could not reuse size-limit action we use for client
since it's very JS centric.
@SevInf SevInf changed the title WIP: report wasm size as a PR comment ci: Report WASM size differences on pull requests Dec 13, 2023
@SevInf SevInf marked this pull request as ready for review December 13, 2023 15:04
@SevInf SevInf requested a review from a team as a code owner December 13, 2023 15:04
@SevInf SevInf requested review from ospfranco and laplab and removed request for a team December 13, 2023 15:04
@SevInf
Copy link
Member Author

SevInf commented Dec 13, 2023

Ignore +23 MB increase in the comment, it was artificially inflated to test the action. Now, action does not run because of paths-ignore. We can temporary comment them out and re-run the action for peace of mind.

Copy link

codspeed-hq bot commented Dec 13, 2023

CodSpeed Performance Report

Merging #4561 will not alter performance

Comparing wasm-size-pr-comment (a495650) with main (762b2b2)

Summary

✅ 11 untouched benchmarks

@SevInf SevInf requested review from miguelff, Jolg42 and jkomyno and removed request for ospfranco and laplab December 13, 2023 15:14
.github/workflows/wasm-size.yml Show resolved Hide resolved
@miguelff
Copy link
Contributor

miguelff commented Dec 13, 2023

This looks good once at least Joel's comment on the path ignore is addressed.

Copy link
Contributor

@Jolg42 Jolg42 left a comment

Choose a reason for hiding this comment

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

Nice!

@SevInf SevInf merged commit 3f521fb into main Dec 13, 2023
6 checks passed
@SevInf SevInf deleted the wasm-size-pr-comment branch December 13, 2023 15:33
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