-
Notifications
You must be signed in to change notification settings - Fork 24
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 gatsby build pipeline #196
base: master
Are you sure you want to change the base?
Conversation
And unused gatsby yarn
Also remove obsolete CNAME
And update package scripts
Use Node 14 in deploy-pages for now
@noahsark769 You can see the workflow run details. It's rather slow and emits loads of warnings. but seems to work ok otherwise. You probably know better if all the warnings are present the whole time locally too. As this won't be run too often I don't care that much about output sizes, caches and 2-3mins build time as long as these legacy versions work, run and build;) so you can just forget about deploys and have them always published on master. 🚀 |
1f5aaaf
to
8e934e7
Compare
Being in ./web makes all steps unhappy looking for default locations
gh-pages
deploy path writing to branch by instead publishing the build artifacts directly to the (existing)github-pages
environment:gh-pages
branch can be removed as it's not used anymore;github-pages
deployment exists from the old implicit runs, no need to create…gh-pages
to publish, needs to be changed tomaster
in repo settings (or removed altogether) [docs]fbjs
dep that should've been loaded by somereact-dom
deps however was missing 🤷gh-pages
dep (as publishing from local dev is not needed anymore), incl. the run commands.