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: Generate the slim build on grunt & run compare_size on it #5291

Merged
merged 1 commit into from
Jul 10, 2023

Conversation

mgol
Copy link
Member

@mgol mgol commented Jul 10, 2023

Summary

Summary of the changes:

  • expand node_smoke_tests to test the full & slim builds
  • run compare_size on all built minified files; don't run it anymore on unminified files where they don't provide lots of value

The main goal of this change is to make it easier to compare sizes of both the full & slim builds between the 3.x-stable & main branches.

Ref gh-5255

(partially cherry-picked from commit 8be4c0e)

Checklist

Summary of the changes:
* expand `node_smoke_tests` to test the full & slim builds
* run `compare_size` on all built minified files; don't run it anymore on
  unminified files where they don't provide lots of value

The main goal of this change is to make it easier to compare sizes of both the
full & slim builds between the `3.x-stable` & `main` branches.

Ref jquerygh-5255

(partially cherry-picked from commit 8be4c0e)
@mgol mgol added Needs review Build Discuss in Meeting Reserved for Issues and PRs that anyone would like to discuss in the weekly meeting. labels Jul 10, 2023
@mgol mgol self-assigned this Jul 10, 2023
@mgol mgol removed Discuss in Meeting Reserved for Issues and PRs that anyone would like to discuss in the weekly meeting. Needs review labels Jul 10, 2023
@mgol mgol added this to the 3.7.1 milestone Jul 10, 2023
@mgol mgol merged commit 763ade6 into jquery:3.x-stable Jul 10, 2023
11 checks passed
@mgol mgol deleted the 3.x-generate-slim-on-build branch July 10, 2023 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

2 participants