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

🏗 Partial updates for core devDependencies #39768

Merged

Conversation

danielrozenberg
Copy link
Member

@danielrozenberg danielrozenberg commented Jan 23, 2024

Updating all of the devDependencies together is hard. Splitting this to make it easier.

This PR updates the dependencies starting with r through z, excluding:

  • tempy, typescript, and sinon which requires non-trivial manual changes to update
  • react*, as this package is updated by a separate renovate job

NOTE: this PR causes non-trivial bundle-size increases!

@danielrozenberg danielrozenberg force-pushed the core-devdep-partial-r branch 2 times, most recently from b479b1d to 098ffb4 Compare January 23, 2024 20:53
@danielrozenberg danielrozenberg marked this pull request as ready for review January 24, 2024 18:54
@danielrozenberg
Copy link
Member Author

@erwinmombay requires re-approval because I had to force-push a new commit to fix a merge conflict, and the bundle-size change requires approval on the last commit :)

@danielrozenberg
Copy link
Member Author

@erwinmombay requires re-approval because I had to force-push a new commit to fix a merge conflict, and the bundle-size change requires approval on the last commit :)

I decided to figure out why the bundle size increased so much, looks like there's a bug in terser that was introduced in version 5.16.3 (terser/terser#1421), so I'm pinning this to 5.16.2. This bug has been reported in August 2023 and I'm not sure when to expect a fix

@danielrozenberg
Copy link
Member Author

Looks like even 5.16.2 causes bundle size changes, so reverting the change to terser

@danielrozenberg danielrozenberg merged commit 3dcace4 into ampproject:main Jan 26, 2024
43 checks passed
@danielrozenberg danielrozenberg deleted the core-devdep-partial-r branch January 26, 2024 20:04
eszponder pushed a commit to krzysztofequativ/amphtml that referenced this pull request Apr 22, 2024
* Partial updates for core devDependencies

* Reduce `terser` back to last-known version that doesn't increase the bundle size

* Reduce `terser` back to current version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants