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

feat: add sourcemap support to NFT #818

Merged
merged 3 commits into from
Nov 18, 2021
Merged

feat: add sourcemap support to NFT #818

merged 3 commits into from
Nov 18, 2021

Conversation

eduardoboucas
Copy link
Member

- Summary

When using NFT, generates a sourcemap for any transpiled files, whenever the nodeSourcemap property is set to true.

- Test plan

Added a new test.

@eduardoboucas eduardoboucas added the type: feature code contributing to the implementation of a feature and/or user facing functionality label Nov 17, 2021

// Asserts that the line/column of the error match the position of the
// original source file, not the transpiled one.
t.true(error.stack.includes(`${filePath}:2:9`))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥

@kodiakhq kodiakhq bot merged commit eddf2a3 into main Nov 18, 2021
@kodiakhq kodiakhq bot deleted the feat/nft-sourcemap branch November 18, 2021 09:58
@github-actions
Copy link
Contributor

⏱ Benchmark results

largeDepsEsbuild: 6.8s

^   6.8s  
│   ┌──┐  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
└───┴──┴──>
     T    
Legend
  • T (current commit): 6.8s

largeDepsNft: 46.2s

^  46.2s  
│   ┌──┐  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
└───┴──┴──>
     T    
Legend
  • T (current commit): 46.2s

largeDepsZisi: 57.1s

^  57.1s  
│   ┌──┐  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
│   |▒▒|  
└───┴──┴──>
     T    
Legend
  • T (current commit): 57.1s

Skn0tt pushed a commit to netlify/build that referenced this pull request May 21, 2024
* feat: add sourcemap support to NFT

* chore: add comment

Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
automerge type: feature code contributing to the implementation of a feature and/or user facing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants