-
Notifications
You must be signed in to change notification settings - Fork 66
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
Remove plugins with required inputs #57
Comments
Fixed via #58 |
Re-opening, we should also remove: |
Should we send PRs to those plugins to add default values to those plugins? |
I would say - if it's a small PR we can do it ourselves. |
Also related https://github.com/getsentry/sentry-netlify-build-plugin, we just need to open a PR for them to remove the required inputs as they already fallback to env variables. |
@erezrokah Does the Ghost plugin not work when the required env vars are set? |
The Ghost plugin has required inputs so it fails validation in Netlify build when installed from UI. |
Some plugins fail UI installation since the have required inputs:
We should remove them as they fail the build immediately when installed from UI.
The text was updated successfully, but these errors were encountered: