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

fix: Default puppeteer-config to not set executable path unless OS is… #235

Merged
merged 5 commits into from
Aug 17, 2023

Conversation

keonik
Copy link
Owner

@keonik keonik commented Aug 16, 2023

… anything but Windows

closes #234

@keonik keonik merged commit d4e99f0 into alpha Aug 17, 2023
6 checks passed
@keonik keonik deleted the issue-234 branch August 17, 2023 13:36
@github-actions
Copy link

🎉 This PR is included in version 1.10.0-alpha.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

keonik added a commit that referenced this pull request Aug 22, 2023
* feat: Configuration of Mermaid via .js file (#232)

* feat: Enable override of mermaid configuration via a .js file

* fix: Move theme to mermaid config and out of cli extensions

* remove mermaid types stub

* fix theme for readme erd

* fix: Documentation for mermaid configuration and note about theme setting via env variable

* feat: Add disable flag to prisma config options  (#233)

close issue #202 by adding an environment variable friendly approach to disabling the generator

* fix: Default puppeteer-config to not set executable path unless OS is… (#235)

* fix: Default puppeteer-config to not set executable path unless OS is anything but Windows

* attempt to remove setting default executable path to troubleshoot mac os test failure in github actions

* .

* cross-env use for windows

* fix: Move puppeteer args to arm64 darwin only
@github-actions
Copy link

🎉 This PR is included in version 1.12.0-alpha.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

1 participant