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

feat: mirror https://binaries.prisma.sh #261

Merged
merged 1 commit into from
Mar 13, 2021
Merged

Conversation

TooBug
Copy link
Contributor

@TooBug TooBug commented Feb 22, 2021

This change is Reviewable

@codecov
Copy link

codecov bot commented Feb 22, 2021

Codecov Report

Merging #261 (7d24151) into master (060c6ca) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #261   +/-   ##
=======================================
  Coverage   57.81%   57.81%           
=======================================
  Files          19       19           
  Lines         697      697           
  Branches      110      110           
=======================================
  Hits          403      403           
  Misses        294      294           
Impacted Files Coverage Δ
config/index.js 80.00% <ø> (ø)

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 060c6ca...5a01182. Read the comment docs.

Copy link
Member

@fengmk2 fengmk2 left a comment

Choose a reason for hiding this comment

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

+1

@fengmk2 fengmk2 merged commit c1a8667 into cnpm:master Mar 13, 2021
@fengmk2
Copy link
Member

fengmk2 commented Mar 13, 2021

@TooBug 文件是同步了,但是文件夹没有处理好。应该是 ListBucketResult 没有考虑子文件夹的问题
https://cnpmjs.org/mirrors/prisma/2.0.x/7d77a7cd570c9dedb4cdf31a1cbec615cf2e1eda/linux-musl/

@TooBug
Copy link
Contributor Author

TooBug commented Mar 13, 2021

有空我再研究一下,我有点没看懂……另外s3是如何处理翻页的也没看懂,好像只能列出1000个key,代码库中也没看到有处理翻页啥的。

@fengmk2
Copy link
Member

fengmk2 commented Mar 13, 2021

@TooBug 是的,现在都放 github releases 了,所以 s3 的都没去再重构了。

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants