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

docs(v2): update steps in the github-actions section #4876

Merged
merged 3 commits into from
Jun 1, 2021

Conversation

wise-introvert
Copy link
Contributor

Created a warning, asking users to replace email and name in the GitHub action with their email and name.

Motivation

I ran into issues when setting up github actions.

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!)

Related PRs

(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label May 31, 2021
@netlify
Copy link

netlify bot commented May 31, 2021

✔️ [V2]
Built without sensitive environment variables

🔨 Explore the source changes: 60a23d1

🔍 Inspect the deploy log: https://app.netlify.com/sites/docusaurus-2/deploys/60b60ab5834a6200089f6074

😎 Browse the preview: https://deploy-preview-4876--docusaurus-2.netlify.app

@github-actions
Copy link

github-actions bot commented May 31, 2021

⚡️ Lighthouse report for the changes in this PR:

Category Score
🟠 Performance 64
🟢 Accessibility 97
🟢 Best practices 100
🟢 SEO 100
🟢 PWA 95

Lighthouse ran on https://deploy-preview-4876--docusaurus-2.netlify.app/

@wise-introvert
Copy link
Contributor Author

I just realised that I didn't update current and older version of the documentation 😅. Will do that..

Copy link
Contributor

@HonkingGoose HonkingGoose left a comment

Choose a reason for hiding this comment

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

Some small things to make this better:

  • Capitalize proper noun GitHub
  • Use monospaced font for both terms to be replaced
  • End sentence with a full stop

You can apply the suggestions to fix these things.

@@ -150,6 +150,12 @@ This workflow assumes your documentation resided in `documentation` branch of yo
1. Copy your private key with `xclip -sel clip < ~/.ssh/id_rsa` and paste a GitHub secret with name `GH_PAGES_DEPLOY`. Copy file content if the command line doesn't work for you. Save your secret.
1. Create you [documentation workflow file](https://help.github.com/en/actions/configuring-and-managing-workflows/configuring-a-workflow#creating-a-workflow-file) in `.github/workflows/`. In this example it's `documentation.yml`.

:::warning

Please make sure that you replace actions@github.com with your github email and gh-actions with your name
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Please make sure that you replace actions@github.com with your github email and gh-actions with your name
Please make sure that you replace `actions@github.com` with your GitHub email and `gh-actions` with your name.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done: 0aae91

website/docs/deployment.mdx Outdated Show resolved Hide resolved
@@ -150,6 +150,12 @@ This workflow assumes your documentation resided in `documentation` branch of yo
1. Copy your private key with `xclip -sel clip < ~/.ssh/id_rsa` and paste a GitHub secret with name `GH_PAGES_DEPLOY`. Copy file content if the command line doesn't work for you. Save your secret.
1. Create you [documentation workflow file](https://help.github.com/en/actions/configuring-and-managing-workflows/configuring-a-workflow#creating-a-workflow-file) in `.github/workflows/`. In this example it's `documentation.yml`.

:::warning

Please make sure that you replace actions@github.com with your github email and gh-actions with your name
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Please make sure that you replace actions@github.com with your github email and gh-actions with your name
Please make sure that you replace `actions@github.com` with your GitHub email and `gh-actions` with your name.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done: 0aae91

website/versioned_docs/version-2.0.0-beta.0/deployment.mdx Outdated Show resolved Hide resolved
website/docs/deployment.mdx Outdated Show resolved Hide resolved
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
@slorber
Copy link
Collaborator

slorber commented Jun 1, 2021

thanks

@slorber slorber changed the title docs: update steps in the github-actions section docs(v2): update steps in the github-actions section Jun 1, 2021
@slorber slorber added the pr: documentation This PR works on the website or other text documents in the repo. label Jun 1, 2021
@slorber slorber merged commit 62caaed into facebook:master Jun 1, 2021
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: documentation This PR works on the website or other text documents in the repo.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants