Entry point without TLA for js package

[bbohlender]

Follow up on #1637

TLDR: tooling for TLA is not there yet; An additional entry point without top-level-await is appropriate

• adds ./load entry to js package
• uses .js file extensions to prevent requiring allowImportingTsExtensions

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
yoga-website	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 8, 2024 5:39pm
yoga-website-legacy	❌ Failed (Inspect)			Apr 8, 2024 5:39pm

[NickGerleman]

uses .js file extensions to prevent requiring allowImportingTsExtensions

We are already doing this in the babel transform during packing. TS entry points in package.json are also rewritten.

[NickGerleman]

It looks like the Jest environment didn’t like the extension change.

The model in the repo right now is that when Yoga is published, its package entry point is changed to be written to babel generated JS, along with rewriting imports. But still include original sources as well.

[bbohlender]

Sorry for my confusion

[bbohlender]

again sorry for forgetting to run eslint :)

[facebook-github-bot]

@NickGerleman has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[NickGerleman]

@bbohlender would you be able to give this a quick mention in the package readme?

[bbohlender]

@NickGerleman, for sure! Is that, what you had in mind?

[NickGerleman]

That looks great. Thanks for this PR!

[facebook-github-bot]

@NickGerleman has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[facebook-github-bot]

@NickGerleman has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[facebook-github-bot]

@NickGerleman merged this pull request in facebook/yoga@bbdd1afe596b3f78fa5887410513d9a98b944715.