[Blocked on AWS fix and release] fix: bump aws for package.json#imports tracing#1231
[Blocked on AWS fix and release] fix: bump aws for package.json#imports tracing#1231james-elicx wants to merge 8 commits intomainfrom
package.json#imports tracing#1231Conversation
🦋 Changeset detectedLatest commit: 6ac03c1 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
commit: |
package.json#imports tracing
|
It works! |
We need to figure out if this is specific to your app - maybe it was not working because of missing files and they have to be included. In that case this fix would work as intended. But we should make sure it doesn't negatively affect other apps. @james-elicx any thoughts on this? |
|
I ran all the builds and put the before / after sizes into a table. The only notable size difference is adding mathjax to the playground16 app. Webpack
Turbopack
|
|
Any difference you're seeing will likely be legitimate changes based on the above tables. |
|
Maybe it would we worth investigating why playground16 is +20%. WDYT? |
|
It's jumped because I added a reproduction for the issue with the library in question. If I remove the repro, there's no change. |
Thanks a lot again for looking into this. |
4c4817a to
4549dcb
Compare
package.json#imports tracingpackage.json#imports tracing
ref: opennextjs/opennextjs-aws#1151
fixes #1229