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

add PUPPETEER_DOWNLOAD_BASE_URL for puppeteer@20 #47

Merged
merged 2 commits into from
May 5, 2023

Conversation

elrrrrrrr
Copy link
Member

Add PUPPETEER_DOWNLOAD_BASE_URL environment variable for compatibility with puppeteer@20 ref puppeteer/puppeteer#10131

  • 🆕 Added env config
  • 🤖 Update mockAgent in test case

新增 PUPPETEER_DOWNLOAD_BASE_URL 环境变量,用以适配 puppeteer@20 ref puppeteer/puppeteer#10131

  • 🆕 增加 env 配置
  • 🤖 修改测试用例 mock 方法

@elrrrrrrr elrrrrrrr requested review from fengmk2 and gemwuu May 5, 2023 05:42
@elrrrrrrr elrrrrrrr force-pushed the feat-add-puppeteer-new-env branch from b12c2d5 to dfaa064 Compare May 5, 2023 06:39
@codecov
Copy link

codecov bot commented May 5, 2023

Codecov Report

Merging #47 (156b298) into master (f010757) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #47   +/-   ##
=======================================
  Coverage   74.61%   74.61%           
=======================================
  Files           2        2           
  Lines         193      193           
  Branches       34       34           
=======================================
  Hits          144      144           
  Misses         49       49           

@elrrrrrrr elrrrrrrr changed the title WIP: add new env for puppeteer@20 add PUPPETEER_DOWNLOAD_BASE_URL for puppeteer@20 May 5, 2023
@fengmk2
Copy link
Member

fengmk2 commented May 5, 2023

PUPPETEER_DOWNLOAD_HOST=https://cdn.npmmirror.com/binaries/chrome-for-testing npminstall puppeteer@19
PUPPETEER_DOWNLOAD_HOST=https://cdn.npmmirror.com/binaries/chrome-for-testing npminstall puppeteer@20

@elrrrrrrr 老的 PUPPETEER_DOWNLOAD_HOST 也改掉,已经兼容。

@elrrrrrrr elrrrrrrr force-pushed the feat-add-puppeteer-new-env branch from dfaa064 to 1562846 Compare May 5, 2023 09:03
@@ -58,7 +58,7 @@
"SASS_BINARY_SITE": "https://cdn.npmmirror.com/binaries/node-sass",
"SWC_BINARY_SITE": "https://cdn.npmmirror.com/binaries/node-swc",
"NWJS_URLBASE": "https://cdn.npmmirror.com/binaries/nwjs/v",
"PUPPETEER_DOWNLOAD_HOST": "https://cdn.npmmirror.com/binaries",
"PUPPETEER_DOWNLOAD_HOST": "https://cdn.npmmirror.com/binaries/chrome-for-testing",
Copy link
Member

Choose a reason for hiding this comment

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

@elrrrrrrr PUPPETEER_DOWNLOAD_BASE_URL 也得加上,新版本不会使用 PUPPETEER_DOWNLOAD_HOST 了

Copy link
Member Author

Choose a reason for hiding this comment

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

已添加 👌🏻

@elrrrrrrr elrrrrrrr force-pushed the feat-add-puppeteer-new-env branch from 1562846 to 156b298 Compare May 5, 2023 09:17
@fengmk2 fengmk2 merged commit dec350b into master May 5, 2023
@fengmk2 fengmk2 deleted the feat-add-puppeteer-new-env branch May 5, 2023 09:26
fengmk2 pushed a commit that referenced this pull request May 5, 2023
[skip ci]

## [2.11.0](v2.10.0...v2.11.0) (2023-05-05)

### Features

* add  PUPPETEER_DOWNLOAD_BASE_URL  for puppeteer@20 ([#47](#47)) ([dec350b](dec350b))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants