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

fix: build-tsConfig #393

Merged
merged 3 commits into from
Feb 20, 2020
Merged

fix: build-tsConfig #393

merged 3 commits into from
Feb 20, 2020

Conversation

echosoar
Copy link
Member

fix build-tsConfig

@echosoar echosoar changed the title fix build-tsConfig fix: build-tsConfig Feb 19, 2020
@codecov-io
Copy link

codecov-io commented Feb 19, 2020

Codecov Report

Merging #393 into 2.x will increase coverage by 0.31%.
The diff coverage is 68.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##              2.x     #393      +/-   ##
==========================================
+ Coverage   86.16%   86.48%   +0.31%     
==========================================
  Files          77       77              
  Lines        2046     2057      +11     
  Branches      124      124              
==========================================
+ Hits         1763     1779      +16     
+ Misses        261      256       -5     
  Partials       22       22
Impacted Files Coverage Δ
packages/midway-bin/lib/cmd/build.js 84.05% <68.42%> (+5.31%) ⬆️

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 2d398fb...c077cb5. Read the comment docs.

const cwd = path.join(__dirname, '../../fixtures/ts-dir-no-config');
await builder.run({
cwd,
argv: {
Copy link
Contributor

Choose a reason for hiding this comment

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

命令行参数是 object 的话会如何使用呢?

Copy link
Member Author

Choose a reason for hiding this comment

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

命令行只支持传入tsconfig文件路径 (project)

Copy link
Member

Choose a reason for hiding this comment

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

命令行是不是得判断下,如果是字符串,parse 一下?

@czy88840616 czy88840616 merged commit fb451b4 into 2.x Feb 20, 2020
@czy88840616 czy88840616 deleted the fix/build-tsConfig branch March 5, 2020 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants