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 Docusaurus v2 CircleCI/GH-pages deployment #2273

Merged
merged 4 commits into from
Oct 29, 2020
Merged

fix Docusaurus v2 CircleCI/GH-pages deployment #2273

merged 4 commits into from
Oct 29, 2020

Conversation

slorber
Copy link
Contributor

@slorber slorber commented Oct 29, 2020

Follow-up of Docusaurus v2 migration so that site can go live.

#2263

Note: this site is published to GH pages of react-native repo, not react-native-website repo, which is kind of weird

Highly recommend to migrate Netlify for everything (build production, host production, PR deploy previews) instead of splitting each task between CircleCI/GH-pages/Netlify. One tool is simpler to manage than 3, technically but also for permissions.
Using GH pages is risky at this scale because we can only have 1 deployment per repo, which means any error can make the production fail, while with Netlify we can have 2 deployments side by side and just change the DNS to perform a migration. Also, Netlify allows many more options like server redirects, server caching headers (without the need to use CloudFlare on top)

@netlify
Copy link

netlify bot commented Oct 29, 2020

Deploy preview for react-native ready!

Built with commit 14249ee

https://deploy-preview-2273--react-native.netlify.app

@@ -60,6 +60,7 @@ jobs:
- restore_cache_checkout
- run_yarn
- run: yarn ci-check
- run: export NODE_OPTIONS=--max_old_space_size=4096
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A site build requires more than default node memory.

Deploy builds the site and use 4go. But tests workflow also build the site, so it should rather use 4go as well.

Copy link
Collaborator

@Simek Simek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for working on this fix! Hope this make V2 website live. 🙂

@Simek Simek merged commit 56e10fe into facebook:master Oct 29, 2020
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.

None yet

3 participants