Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 6 commits into from
Dec 16, 2018
Merged

Conversation

odensc
Copy link
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

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
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
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
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
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
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