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

feat: use typescript 2.7.1 and esModuleInterop #124

Closed
wants to merge 1 commit into from

Conversation

jinwoo
Copy link
Member

@jinwoo jinwoo commented Feb 5, 2018

This PR is both for the generated package.json and gts itself.

TypeScript 2.7.1 added --esModuleInterop for better ES module
support. And for that, we need to turn on allowSyntheticDefaultImports
too.

Also tslint must be upgraded to 5.9.1 for it to compile with
esModuleInterop.

@jinwoo jinwoo requested review from ofrobots and a team February 5, 2018 23:19
@codecov-io
Copy link

codecov-io commented Feb 5, 2018

Codecov Report

Merging #124 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #124   +/-   ##
=======================================
  Coverage   96.63%   96.63%           
=======================================
  Files          10       10           
  Lines         327      327           
  Branches       19       19           
=======================================
  Hits          316      316           
  Misses         11       11
Impacted Files Coverage Δ
test/test-kitchen.ts 100% <100%> (ø) ⬆️
src/util.ts 94.11% <100%> (ø) ⬆️
test/fixtures.ts 100% <100%> (ø) ⬆️

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 3234014...90942cd. Read the comment docs.

@ofrobots
Copy link
Contributor

ofrobots commented Feb 5, 2018

Typescript 2.7.x would be a semver major change; so let's no land this now. It would be okay landing this onto a next branch though.

@jinwoo
Copy link
Member Author

jinwoo commented Feb 6, 2018

@ofrobots Yes, you're right that this is a breaking change. What about releasing 0.6.0 after this change? I'm fine with landing onto next if it's not a good idea.

BTW I think we need a release for #114.

[BREAKING CHANGE]

This PR is both for the generated `package.json` and `gts` itself.

TypeScript 2.7.1 added `--esModuleInterop` for better ES module
support. And for that, we need to turn on `allowSyntheticDefaultImports`
too.

Also `tslint` must be upgraded to 5.9.1 for it to compile with
`esModuleInterop`.
@jinwoo
Copy link
Member Author

jinwoo commented Feb 6, 2018

Closing this. New PR in #125.

@jinwoo jinwoo closed this Feb 6, 2018
@jinwoo jinwoo deleted the esmoduleinterop branch February 7, 2018 01:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants