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: use configuration files #9140

Merged
merged 1 commit into from Oct 21, 2022
Merged

feat: use configuration files #9140

merged 1 commit into from Oct 21, 2022

Conversation

jrandolf
Copy link
Contributor

@jrandolf jrandolf commented Oct 19, 2022

This PR adds configurations files to puppeteer's methods for configuration. Under the hood, puppeteer relies on https://www.npmjs.com/package/cosmiconfig which resolves several formats of configuration:

  • a puppeteer property in package.json
  • a .puppeteerrc file in JSON or YAML format
  • a .puppeteerrc.json, .puppeteerrc.yaml, .puppeteerrc.yml, .puppeteerrc.js, or .puppeteerrc.cjs file
  • a puppeteer.config.js or puppeteer.config.cjs CommonJS module exporting an object

Documentation will be added later.

Fixed: #9128

@jrandolf jrandolf requested a review from OrKoN October 19, 2022 16:02
@jrandolf jrandolf force-pushed the configuration branch 11 times, most recently from 30f8726 to 06b308e Compare October 20, 2022 12:01
packages/puppeteer-core/src/node/BrowserFetcher.ts Outdated Show resolved Hide resolved
packages/puppeteer-core/src/node/ChromeLauncher.ts Outdated Show resolved Hide resolved
packages/puppeteer-core/src/node/ChromeLauncher.ts Outdated Show resolved Hide resolved
packages/puppeteer/src/PuppeteerConfiguration.ts Outdated Show resolved Hide resolved
packages/puppeteer/src/PuppeteerConfiguration.ts Outdated Show resolved Hide resolved
packages/puppeteer/src/node/install.ts Outdated Show resolved Hide resolved
@jrandolf jrandolf force-pushed the configuration branch 6 times, most recently from b4d7f38 to 69ae29c Compare October 20, 2022 22:54
@jrandolf jrandolf force-pushed the configuration branch 5 times, most recently from 79a0eb5 to ea05c44 Compare October 21, 2022 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Monorepo breaks build for Cloud Functions
3 participants