Skip to content
This repository has been archived by the owner on Aug 22, 2023. It is now read-only.

feat: use esModuleInterop when defined (fixes #179) #61

Merged
merged 1 commit into from
Nov 7, 2018
Merged

feat: use esModuleInterop when defined (fixes #179) #61

merged 1 commit into from
Nov 7, 2018

Conversation

mfellner
Copy link
Contributor

@mfellner mfellner commented Nov 6, 2018

Fixes oclif/oclif#179.

  • Added esModuleInterop to TSConfig.
  • Added ts-node.test.ts

This pull request updates ts-node.ts to use the esModuleInterop of loaded tsconfig.json files. It should now be possible to write oclif-based CLIs in TypeScript projects with esModuleInterop set to true.

@salesforce-cla
Copy link

salesforce-cla bot commented Nov 6, 2018

Thanks for the contribution! Before we can merge this, we need @mfellner to sign the Salesforce.com Contributor License Agreement.

@codecov
Copy link

codecov bot commented Nov 6, 2018

Codecov Report

Merging #61 into master will increase coverage by 0.93%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #61      +/-   ##
==========================================
+ Coverage   68.69%   69.62%   +0.93%     
==========================================
  Files           7        7              
  Lines         428      428              
  Branches      116      116              
==========================================
+ Hits          294      298       +4     
+ Misses         84       83       -1     
+ Partials       50       47       -3
Impacted Files Coverage Δ
src/ts-node.ts 83.67% <ø> (+8.16%) ⬆️

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 8bf2287...43ef204. Read the comment docs.

Copy link
Contributor

@jdx jdx left a comment

Choose a reason for hiding this comment

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

thanks for the tests!! 🤘

@jdx jdx merged commit 5778618 into oclif:master Nov 7, 2018
jdx pushed a commit that referenced this pull request Nov 7, 2018
# [1.9.0](v1.8.8...v1.9.0) (2018-11-07)

### Features

* use esModuleInterop when defined (fixes [#179](#179)) ([#61](#61)) ([5778618](5778618))
@oclif-bot
Copy link
Contributor

🎉 This PR is included in version 1.9.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@mfellner mfellner deleted the 179-es-module-interop branch November 8, 2018 09:08
martijnwalraven added a commit to apollographql/apollo-tooling that referenced this pull request Mar 20, 2019
See oclif/config#61

Unfortunately, this fix doesn't interpret `extends` in `tsconfig.json`, so I had to add an explicit `"esModuleInterop": true`.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants