search

cnpm / cnpmcore

Private NPM Registry for Enterprise
https://npmmirror.com
MIT License
607 stars 80 forks source link

fix: noImplicitAny ts #568

Closed elrrrrrrr closed 1 year ago

elrrrrrrr commented 1 year ago

Attempted to apply the noImplicitAny, parameter types should be specified unless any is manually declared.

  • 🐞 Fixed an issue in bugVersionAdvice where AbbreviatedManifest was being set abnormally.
  • 🤖 Added index.d.ts to store declarations for dependencies without types.

  • 🤔 skipLibCheck has no effect on leoric for now, so it cannot be enabled temporarily.

    尝试应用 noImplicitAny 配置，除非手动声明 any，否则需要指定参数类型

  • 🐞 修复 bugVersionAdvice 中，AbbreviatedManifest 设置异常
  • 🤖 添加 index.d.ts 存放无类型依赖声明
  • 🤔 skipLibCheck 对 leoric 失效，暂时无法开启

image

socket-security[bot] commented 1 year ago

New dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
@types/mime-types 2.1.1 None +0 3.85 kB types
@types/npm-package-arg 6.1.1 None +0 7.92 kB types
@types/lodash 4.14.196 None +0 863 kB types
codecov[bot] commented 1 year ago

Codecov Report

Merging #568 (1a25928) into master (ab2fde7) will decrease coverage by 0.04%. Report is 24 commits behind head on master. The diff coverage is 98.00%.

@@            Coverage Diff             @@
##           master     #568      +/-   ##
==========================================
- Coverage   97.02%   96.99%   -0.04%     
==========================================
  Files         174      175       +1     
  Lines       16588    16752     +164     
  Branches     2177     2200      +23     
==========================================
+ Hits        16095    16248     +153     
- Misses        493      504      +11
Files Changed Coverage Δ
app/port/webauth/WebauthController.ts 85.59% <78.94%> (+0.53%) :arrow_up:
app/common/PackageUtil.ts 97.61% <90.90%> (-1.42%) :arrow_down:
app/common/FileUtil.ts 100.00% <100.00%> (ø)
app/common/UserUtil.ts 100.00% <100.00%> (ø)
app/common/adapter/CacheAdapter.ts 100.00% <100.00%> (ø)
app/common/adapter/NPMRegistry.ts 100.00% <100.00%> (ø)
app/common/adapter/binary/AbstractBinary.ts 94.17% <100.00%> (+0.11%) :arrow_up:
app/common/adapter/binary/NodePreGypBinary.ts 98.69% <100.00%> (+<0.01%) :arrow_up:
app/common/adapter/binary/PlaywrightBinary.ts 99.26% <100.00%> (+0.03%) :arrow_up:
...on/adapter/changesStream/CnpmjsorgChangesStream.ts 100.00% <100.00%> (ø)
... and 29 more

... and 2 files with indirect coverage changes