Skip to content

Support for symlinks in project references#1136

Merged
johnnyreilly merged 15 commits intomasterfrom
symLinks
Jul 8, 2020
Merged

Support for symlinks in project references#1136
johnnyreilly merged 15 commits intomasterfrom
symLinks

Conversation

@sheetalkamat
Copy link
Copy Markdown
Contributor

Fixes #1114

@johnnyreilly
Copy link
Copy Markdown
Member

Nice work @sheetalkamat!

I can see there's a bunch of failing tests which you're no doubt looking into. Having had a quick look it seems that TypeScript <= 2.6 tests are failing.

I've been wondering about dropping support for older versions of TypeScript; maybe making the minimum supported version TypeScript 3.6. It would speed up our CI considerably as we'd have a much smaller test matrix to run in Travis / AppVeyor. Users on older versions of TypeScript would still have older versions of ts-loader they could use.

If it makes your life easier, then do feel free to consider doing that in this PR. TypeScript 2.6 was a long time ago!

@johnnyreilly
Copy link
Copy Markdown
Member

johnnyreilly commented Jul 4, 2020

From the looks of it this is done @sheetalkamat? Thanks so much for your work! @andrewbranch would you be able to give this a review when you get a chance please?

I've also upgraded the version in the package.json to 8.0.0 and put an entry in the CHANGELOG.md - so that when we're ready to ship we can just go go go 😄

@andrewbranch
Copy link
Copy Markdown
Contributor

Yep, I will do it this week if not today. 🚀

@@ -0,0 +1 @@
export const getMeaningOfLife2 = () => 45; No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

45?! 😱

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

That's a sick mind right there 😉

@johnnyreilly johnnyreilly merged commit 4ad247a into master Jul 8, 2020
@johnnyreilly
Copy link
Copy Markdown
Member

Shipping now https://github.com/TypeStrong/ts-loader/releases/tag/v8.0.0

Thanks for all your work @sheetalkamat and thanks for the review @andrewbranch!

@sheetalkamat sheetalkamat deleted the symLinks branch July 8, 2020 21:48
jedster1111 added a commit to jedster1111/ts-loader-project-references that referenced this pull request Jul 10, 2020
Proved yarn workspaces, project references & ts-loader work together.
Fixed in  TypeStrong/ts-loader#1136
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.

Output while using project references is using the previous dependent package output instead of the current

3 participants