-
Notifications
You must be signed in to change notification settings - Fork 450
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 ability to resolve "extends" directive in tsconfig.json #98
Conversation
Travis breaks for some other reason I think..? |
@emilniklas Thanks for the PR!. Yes, there's an issue open about randomly failing CI tests - #89 The PR looks good to me barring the small change I've suggested |
}); | ||
}); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't it be better to use constants from typescript that represent each of these values instead of numbers?
@kulshekhar Okay! I just copied the test above. I'll go in and change it everywhere. |
Thanks again @emilniklas 😄 |
@kulshekhar any idea when this will get released? |
to publish changes merged in #98
@wmonk done! |
Woohoo, thanks! |
More info: https://www.typescriptlang.org/docs/handbook/tsconfig-json.html#configuration-inheritance-with-extends