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

feat: add doctor command to check potential problems #56

Merged
merged 10 commits into from
Jul 29, 2022

Conversation

PeachScript
Copy link
Member

@PeachScript PeachScript commented Jul 29, 2022

添加 father doctor 命令以检查项目的潜在问题,支持的规则参考: umijs/father#608

@codecov
Copy link

codecov bot commented Jul 29, 2022

Codecov Report

Merging #56 (08b7de8) into master (e60eef0) will increase coverage by 0.65%.
The diff coverage is 98.91%.

❗ Current head 08b7de8 differs from pull request most recent head f5bf66a. Consider uploading reports for the commit f5bf66a to get more accurate results

@@            Coverage Diff             @@
##           master      umijs/father-next#56      +/-   ##
==========================================
+ Coverage   95.05%   95.70%   +0.65%     
==========================================
  Files          33       43      +10     
  Lines         889     1070     +181     
  Branches      197      235      +38     
==========================================
+ Hits          845     1024     +179     
- Misses         43       45       +2     
  Partials        1        1              
Impacted Files Coverage Δ
src/prebundler/config.ts 100.00% <ø> (ø)
src/preset.ts 100.00% <ø> (ø)
src/types.ts 100.00% <ø> (ø)
src/doctor/rules/PACK_FILES_MISSING.ts 89.47% <89.47%> (ø)
src/builder/bundless/index.ts 90.58% <100.00%> (-0.11%) ⬇️
src/builder/config.ts 98.48% <100.00%> (ø)
src/commands/doctor.ts 100.00% <100.00%> (ø)
src/constants.ts 100.00% <100.00%> (ø)
src/doctor/index.ts 100.00% <100.00%> (ø)
src/doctor/parser.ts 100.00% <100.00%> (ø)
... and 7 more

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 e60eef0...f5bf66a. Read the comment docs.

@PeachScript PeachScript marked this pull request as ready for review July 29, 2022 06:55
@PeachScript PeachScript changed the title feat: add doctor command to check usual problems feat: add doctor command to check potential problems Jul 29, 2022
@PeachScript PeachScript merged commit f746159 into master Jul 29, 2022
@delete-merged-branch delete-merged-branch bot deleted the feature/doctor branch July 29, 2022 07:07
@PeachScript PeachScript mentioned this pull request Aug 1, 2022
71 tasks
PeachScript added a commit to umijs/father that referenced this pull request Aug 26, 2022
…xt#56)

* feat: add basic doctor command

* feat: doctor support regular checkup

* feat: add regular checkup rules

* feat: doctor support source checkup

* feat: add source checkup rules

* feat: doctor support source imports checkup

* feat: add source imports checkup rules

* test: add cases for doctor command & rules

* test: correct case

* docs: add simple doctor command description
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant