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

feat(v2): add nojekyll file to static folder for all templates #2902

Merged
merged 1 commit into from
Jun 8, 2020
Merged

feat(v2): add nojekyll file to static folder for all templates #2902

merged 1 commit into from
Jun 8, 2020

Conversation

teikjun
Copy link
Contributor

@teikjun teikjun commented Jun 7, 2020

Motivation

This PR adds a .nojekyll file to the static directory for all templates found in packages/docusaurus-init/templates.

This automatically disables Jekyll when publishing on GitHub Pages. This PR closes #2892.

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)

For each template:

  1. cd into the template and run yarn build
  2. Check that the static directory contains an empty .nojekyll file

Related PRs

Might need to update https://v2.docusaurus.io/docs/next/static-assets/#caveats

@teikjun teikjun requested a review from yangshun as a code owner June 7, 2020 19:19
@docusaurus-bot
Copy link
Contributor

Deploy preview for docusaurus-2 ready!

Built with commit 2e99722

https://deploy-preview-2902--docusaurus-2.netlify.app

@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Jun 7, 2020
Copy link
Contributor

@lex111 lex111 left a comment

Choose a reason for hiding this comment

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

I think it would be nice to actualize the green block quote on this page.

https://v2.docusaurus.io/docs/deployment/#docusaurusconfigjs-settings

Copy link
Contributor

@yangshun yangshun 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!

@yangshun
Copy link
Contributor

yangshun commented Jun 8, 2020

I think it would be nice to actualize the green block quote on this page.

@lex111 what does this mean?

@yangshun yangshun merged commit 66716d1 into facebook:master Jun 8, 2020
@slorber slorber added the pr: bug fix This PR fixes a bug in a past release. label Jun 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Signed Facebook CLA pr: bug fix This PR fixes a bug in a past release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automatically add a .nojekyll file to the static directory when Docusaurus is installed
6 participants