Skip to content

Build: Report compressed sizes in compare_size#2248

Merged
mgol merged 3 commits into
jquery:1-13-stablefrom
mgol:compare-size
May 20, 2024
Merged

Build: Report compressed sizes in compare_size#2248
mgol merged 3 commits into
jquery:1-13-stablefrom
mgol:compare-size

Conversation

@mgol
Copy link
Copy Markdown
Member

@mgol mgol commented May 10, 2024

Just like it has always worked in Core. This will help with size comparisons
between 1.13 & 1.14.

After this PR:

$ grunt sizer 
Running "requirejs:js" (requirejs) task

Running "uglify:main" (uglify) task
>> 1 file created 549 kB → 267 kB

Running "compare_size:all" (compare_size) task
   raw     gz Sizes                                                            
549319 128757 dist/jquery-ui.js                                                
266710  69612 dist/jquery-ui.min.js                                            

Done.

Before, only the less interesting raw sizes were reported.

main version: #2254

@mgol mgol added this to the 1.14.0 milestone May 10, 2024
@mgol mgol requested a review from fnagel May 10, 2024 13:59
@mgol mgol self-assigned this May 10, 2024
@mgol mgol changed the base branch from main to 1-13-stable May 10, 2024 14:00
@mgol mgol changed the title Build: Run tests on jQuery 3.7.1, not 3.7.0 Build: Report compressed sizes in compare_size May 10, 2024
Just like it has always worked in Core. This will help with size comparisons
between 1.13 & 1.14.

After this PR:

```
$ grunt sizer
Running "requirejs:js" (requirejs) task

Running "uglify:main" (uglify) task
>> 1 file created 549 kB → 267 kB

Running "compare_size:all" (compare_size) task
   raw     gz Sizes
549319 128757 dist/jquery-ui.js
266710  69612 dist/jquery-ui.min.js

Done.
```

Before, only the less interesting `raw` sizes were reported.
@mgol mgol requested a review from timmywil May 15, 2024 16:14
Comment thread Gruntfile.js Outdated
@mgol mgol requested a review from timmywil May 18, 2024 22:27
@mgol mgol removed this from the 1.14.0 milestone May 19, 2024
mgol added a commit that referenced this pull request May 20, 2024
Just like it has always worked in Core. This will help with size comparisons
between 1.13 & 1.14.

After this PR:

```
$ grunt sizer
Running "requirejs:js" (requirejs) task

Running "uglify:main" (uglify) task
>> 1 file created 549 kB → 267 kB

Running "compare_size:all" (compare_size) task
   raw     gz Sizes
549319 128757 dist/jquery-ui.js
266710  69612 dist/jquery-ui.min.js

Done.
```

Before, only the less interesting `raw` sizes were reported.

Closes gh-2254
Ref gh-2248
@mgol mgol merged commit f017426 into jquery:1-13-stable May 20, 2024
@mgol mgol deleted the compare-size branch May 20, 2024 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants