Skip to content

Add support for tsconfig paths. (fixes #150)#151

Merged
rauchg merged 6 commits intovercel:masterfrom
odensc:master
Dec 16, 2018
Merged

Add support for tsconfig paths. (fixes #150)#151
rauchg merged 6 commits intovercel:masterfrom
odensc:master

Conversation

@odensc
Copy link
Copy Markdown
Contributor

@odensc odensc commented Dec 13, 2018

This PR adds support for the paths option in tsconfig, using tsconfig-paths-webpack-plugin.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Dec 13, 2018

Codecov Report

Merging #151 into master will increase coverage by 0.66%.
The diff coverage is 95%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #151      +/-   ##
==========================================
+ Coverage   76.82%   77.49%   +0.66%     
==========================================
  Files          11       11              
  Lines         492      511      +19     
==========================================
+ Hits          378      396      +18     
- Misses        114      115       +1
Impacted Files Coverage Δ
src/index.js 93.47% <95%> (+0.32%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 804ee91...d80b75c. Read the comment docs.

@rauchg
Copy link
Copy Markdown
Member

rauchg commented Dec 13, 2018

@odensc I really cannot accept this without tests. Any chance you can add a few? Thank you!

@odensc
Copy link
Copy Markdown
Contributor Author

odensc commented Dec 14, 2018

@rauchg Sure, was gonna add tests but couldn't figure out how to put TypeScript tests in with the current structure.
index.test.js looks specifically for input.js files - should I have it look for both .js and .ts, or add a new folder for TS tests, or...?

@rauchg
Copy link
Copy Markdown
Member

rauchg commented Dec 14, 2018

I believe we already have a typescript test in there! request-ts.ts, which tests bundling request

@odensc
Copy link
Copy Markdown
Contributor Author

odensc commented Dec 16, 2018

@rauchg I couldn't actually find that file in the repo - until 804ee91 there were no .ts files at all. I've added a couple tests though, let me know how that looks. :)

@rauchg rauchg merged commit 4c227fb into vercel:master Dec 16, 2018
@rauchg
Copy link
Copy Markdown
Member

rauchg commented Dec 17, 2018

My bad @odensc – I'd totally forgotten to push my .ts integration test:
d4d0ead

Would you mind adding another one for paths?

guybedford added a commit that referenced this pull request Dec 17, 2018
rauchg added a commit that referenced this pull request Dec 17, 2018
rauchg added a commit that referenced this pull request Dec 17, 2018
* Revert "fix minify default (#168)"

This reverts commit c289b28.

* Revert "Reduce asset emission and handle unexpected core assets (#166)"

This reverts commit b83e921.

* Revert "use graceful-fs (#167)"

This reverts commit 4ab1f82.

* Revert "fix ignore"

This reverts commit e6ab392.

* Revert "add typescript config for test"

This reverts commit fa95d0b.

* Revert "0.6.0"

This reverts commit 2f12e50.

* Revert "add typescript test"

This reverts commit d4d0ead.

* Revert "Add support for tsconfig `paths`. (fixes #150) (#151)"

This reverts commit 4c227fb.
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.

3 participants