Skip to content

refactor(config): set default value CommonJS after resolving tsconfig#2280

Merged
ahnpnl merged 1 commit intokulshekhar:masterfrom
ahnpnl:default-module
Jan 15, 2021
Merged

refactor(config): set default value CommonJS after resolving tsconfig#2280
ahnpnl merged 1 commit intokulshekhar:masterfrom
ahnpnl:default-module

Conversation

@ahnpnl
Copy link
Copy Markdown
Collaborator

@ahnpnl ahnpnl commented Jan 15, 2021

Summary

Set default module CommonJS after resolving tsconfig

Test plan

Adjusted unit test, green CI

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

N.A.

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 488018417

  • 0 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.01%) to 93.668%

Totals Coverage Status
Change from base Build 487936243: 0.01%
Covered Lines: 948
Relevant Lines: 978

💛 - Coveralls

@coveralls
Copy link
Copy Markdown

coveralls commented Jan 15, 2021

Pull Request Test Coverage Report for Build 488018417

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.01%) to 93.668%

Totals Coverage Status
Change from base Build 487936243: 0.01%
Covered Lines: 948
Relevant Lines: 978

💛 - Coveralls

@ahnpnl ahnpnl marked this pull request as ready for review January 15, 2021 13:46
@ahnpnl ahnpnl requested a review from kulshekhar as a code owner January 15, 2021 13:46
@ahnpnl ahnpnl merged commit 067b653 into kulshekhar:master Jan 15, 2021
@ahnpnl ahnpnl deleted the default-module branch January 15, 2021 14:47
@ahnpnl ahnpnl added this to the ts-jest v27 milestone Jan 15, 2021
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