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

feat: Sync libpg-query binary #557

Merged
merged 3 commits into from
Jul 27, 2023
Merged

Conversation

wandergis
Copy link
Contributor

No description provided.

@elrrrrrrr
Copy link
Member

@wandergis 感谢贡献,麻烦补充一下测试用例?

可以参照 test/common/adapter/binary/NodePreGypBinary.test.ts 添加下

@codecov
Copy link

codecov bot commented Jul 26, 2023

Codecov Report

Merging #557 (b3b55a0) into master (4455295) will decrease coverage by 0.06%.
Report is 1 commits behind head on master.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #557      +/-   ##
==========================================
- Coverage   97.03%   96.97%   -0.06%     
==========================================
  Files         174      174              
  Lines       16674    16681       +7     
  Branches     2194     2190       -4     
==========================================
- Hits        16179    16177       -2     
- Misses        495      504       +9     
Files Changed Coverage Δ
config/binaries.ts 100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

@wandergis
Copy link
Contributor Author

@wandergis 感谢贡献,麻烦补充一下测试用例?

可以参照 test/common/adapter/binary/NodePreGypBinary.test.ts 添加下

已添加测试用例

@fengmk2 fengmk2 added the enhancement New feature or request label Jul 26, 2023
@fengmk2
Copy link
Member

fengmk2 commented Jul 26, 2023

@wandergis npm run lint --fix 修复一下

@wandergis
Copy link
Contributor Author

@wandergis npm run lint --fix 修复一下

done

@fengmk2 fengmk2 changed the title 📦 NEW: Sync libpg-query binary feat: Sync libpg-query binary Jul 27, 2023
@fengmk2 fengmk2 merged commit 8556b5f into cnpm:master Jul 27, 2023
7 of 9 checks passed
fengmk2 pushed a commit that referenced this pull request Jul 27, 2023
[skip ci]

## [3.39.0](v3.38.2...v3.39.0) (2023-07-27)

### Features

* Sync libpg-query binary ([#557](#557)) ([8556b5f](8556b5f))
assert(item.size === '-');
assert(
item.url ===
'https://supabase-public-artifacts-bucket.s3.amazonaws.com/libpg-query/queryparser-v13.2.1-node-v108-darwin-arm64.tar.gz',
Copy link
Member

Choose a reason for hiding this comment

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

@wandergis 这个下载地址不对,应该是 https://supabase-public-artifacts-bucket.s3.amazonaws.com/libpg-query-node/queryparser-v13.1.2-node-v83-darwin-arm64.tar.gz

路径是 libpg-query-node ,而不是 libpg-query

Copy link
Contributor Author

Choose a reason for hiding this comment

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

我马上验证一下

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@fengmk2 #559 重新提了个pr,修改了一下NodePreGypBinary.ts的部分逻辑,看下是否正确?

fengmk2 added a commit to cnpm/binary-mirror-config that referenced this pull request Jul 31, 2023
fengmk2 added a commit to cnpm/binary-mirror-config that referenced this pull request Jul 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants