Skip to content
This repository has been archived by the owner on May 22, 2024. It is now read-only.

fix: disable ESM transpilation in NFT #786

Merged
merged 2 commits into from
Nov 2, 2021
Merged

fix: disable ESM transpilation in NFT #786

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
461cef1
fix: disable ESM transpilation in NFT
eduardoboucas Nov 2, 2021
7dd8c2c
refactor: remove transpilation
eduardoboucas Nov 2, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
3 changes: 1 addition & 2 deletions src/runtimes/node/bundlers/nft/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ const bundle: BundleFunction = async ({
includedFiles,
includedFilesBasePath || basePath,
)
const { paths: dependencyPaths, rewrites } = await traceFilesAndTranspile({
const { paths: dependencyPaths } = await traceFilesAndTranspile({
basePath: repositoryRoot,
config,
mainFile,
Expand All @@ -48,7 +48,6 @@ const bundle: BundleFunction = async ({
basePath: getBasePath(dirnames),
inputs: dependencyPaths,
mainFile,
rewrites,
srcFiles,
}
}
Expand Down
4 changes: 2 additions & 2 deletions tests/main.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -410,7 +410,7 @@ testMany(

testMany(
'Can bundle functions with `.js` extension using ES Modules and feature flag ON',
['bundler_esbuild', 'bundler_default', 'bundler_nft'],
['bundler_esbuild', 'bundler_default', 'todo:bundler_nft'],
async (options, t) => {
const opts = merge(options, { featureFlags: { defaultEsModulesToEsbuild: true } })

Expand All @@ -423,7 +423,7 @@ testMany(

testMany(
'Can bundle functions with `.js` extension using ES Modules and feature flag OFF',
['bundler_esbuild', 'bundler_default', 'bundler_nft'],
['bundler_esbuild', 'bundler_default', 'todo:bundler_nft'],
async (options, t) => {
const fixtureName = 'local-require-esm'
const opts = merge(options, {
Expand Down