Skip to content

Conversation

@ilteoood
Copy link
Contributor

After some benchmarks, I see that there are no gains doing it, so:

Revert of #2341

@ilteoood ilteoood requested review from 2heal1 and zhoushaw April 30, 2024 06:55
@changeset-bot
Copy link

changeset-bot bot commented Apr 30, 2024

🦋 Changeset detected

Latest commit: c4e3da4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@module-federation/native-federation-typescript Patch

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

@netlify
Copy link

netlify bot commented Apr 30, 2024

Deploy Preview for module-federation-docs ready!

Name Link
🔨 Latest commit c4e3da4
🔍 Latest deploy log https://app.netlify.com/sites/module-federation-docs/deploys/6634773465dd9400085959bb
😎 Deploy Preview https://deploy-preview-2428--module-federation-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ilteoood ilteoood force-pushed the revert/nft-speedup branch from 825d9e6 to 40dd0b4 Compare April 30, 2024 06:57
Co-authored-by: zhouxiao.shaw <zhouxiao.shaw@bytedance.com>
@ilteoood ilteoood force-pushed the revert/nft-speedup branch from 40dd0b4 to 6cddb66 Compare April 30, 2024 06:58
@ilteoood ilteoood requested a review from ScriptedAlchemy May 1, 2024 06:32
@ilteoood
Copy link
Contributor Author

ilteoood commented May 3, 2024

ping @ScriptedAlchemy @zhoushaw

@ilteoood ilteoood merged commit 6f743f0 into main May 4, 2024
@ilteoood ilteoood deleted the revert/nft-speedup branch May 4, 2024 08:17
@zhoushaw zhoushaw mentioned this pull request May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants