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
Feature/workflow updates #11
Conversation
@circlecube Looks like you have some conflicts. Also, should be merging into |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No blockers, but I left some notes on things we can do to simplify/clean up.
- name: Run NPM Install | ||
run: npm install | ||
- name: NPM Install | ||
run: npm install --legacy-peer-deps |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We may not have to have the --legacy-peer-deps
flag. If you can run a normal npm install
without getting any errors/warnings, we don't need it. If you get a notice about missing peer dependencies, then we either need to add those or add this flag. It was easier to add the flag on the Bluehost plugin, but may not be required on the other plugins.
Update workflows
Addresses #10