-
Notifications
You must be signed in to change notification settings - Fork 240
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
Conversation
WASM Size
|
a3c916a
to
a495650
Compare
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.
a495650
to
49639e1
Compare
Ignore +23 MB increase in the comment, it was artificially inflated to test the action. Now, action does not run because of |
CodSpeed Performance ReportMerging #4561 will not alter performanceComparing Summary
|
Co-authored-by: Joël Galeran <[email protected]>
Co-authored-by: Joël Galeran <[email protected]>
This looks good once at least Joel's comment on the path ignore is addressed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
Co-authored-by: Joël Galeran <[email protected]>
Co-authored-by: Joël Galeran <[email protected]>
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.