search

cnpm / cnpmcore

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

fix: publisher info #565

Closed elrrrrrrr closed 1 year ago

elrrrrrrr commented 1 year ago

Fixed an issue with the _npmUser field setting during the initial sync.

  1. 🧶 Fixed _npmUser field setting issue during initial sync, should use displayName.
  2. 🧶 skip diff abbreviated meta for _npmUser
  3. 🤖 Refined some TypeScript definitions, use isEqual to diff metas

修复首次同步时，_npmUser 字段设置异常

  1. 🧶 修复 publisher 匹配，获取逻辑，应当用 displayName 进行匹配
  2. 🧶 精简 meta 信息，跳过比较不存在的 _npmUser 字段
  3. 🤖 调整部分 ts 定义，使用 isEqual 来进行 diff 比较
codecov[bot] commented 1 year ago

Codecov Report

Merging #565 (3b70d86) into master (bcf3547) will increase coverage by 0.00%. Report is 1 commits behind head on master. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #565   +/-   ##
=======================================
  Coverage   96.98%   96.98%           
=======================================
  Files         175      175           
  Lines       16714    16715    +1     
  Branches     2196     2196           
=======================================
+ Hits        16210    16211    +1     
  Misses        504      504
Files Changed Coverage Δ
app/core/service/PackageManagerService.ts 98.86% <100.00%> (ø)
app/core/service/PackageSyncerService.ts 98.21% <100.00%> (-0.01%) :arrow_down:
app/repository/PackageRepository.ts 98.19% <100.00%> (+0.01%) :arrow_up: