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: minor update to "set up fcc locally guidelines" - adding 'closing quotes' to docker set up #38149

Conversation

Hash2C
Copy link
Contributor

@Hash2C Hash2C commented Feb 1, 2020

  • I have read freeCodeCamp's contribution guidelines.
  • My pull request has a descriptive title (not a vague title like Update index.md)
  • My pull request targets the master branch of freeCodeCamp.
  • None of my changes are plagiarized from another source without proper attribution.
  • All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
  • My changes do not use shortened URLs or affiliate links.

On "how-to-setup-freecodecamp-locally.md".
Minor fix, helps new contributors to copy/paste the commands.
Just added some closing " that were missing.
Depending on the Docker configuration, it might be needed to prepend the commands with sudo, I wonder if some "tip" should be included for this case.

Going for the first freeCodeCamp pull request. Also looking for feedback. woohoo!

@gitpod-io
Copy link

gitpod-io bot commented Feb 1, 2020

Comment on lines +294 to +297
| `npm run docker:test -- -c "npm run test"` | Run all JS tests in the system, including client, server, lint and challenge tests. |
| `npm run docker:test -- -c "npm run test:curriculum"` | Run the curriculum test suite. |
| `npm run docker:test -- -c "npm run test:client"` | Run the client test suite. |
| `npm run docker:test -- -c "npm run test:server"` | Run the server test suite. |
Copy link
Contributor Author

@Hash2C Hash2C Feb 1, 2020

Choose a reason for hiding this comment

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

Just exploring. Gourgeous stuff xD

Choose a reason for hiding this comment

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

Just exploring. Gourgeous stuff xD

Thank you for your cleanup

Choose a reason for hiding this comment

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

thanks will test

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it surely is a pleasure to contribute

Copy link
Contributor

@ojeytonwilliams ojeytonwilliams left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@ojeytonwilliams ojeytonwilliams merged commit 0e4a140 into freeCodeCamp:master Feb 1, 2020
@ojeytonwilliams
Copy link
Contributor

@Hash2C Congrats on your first pull request (PR)! 🎉

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝

@Hash2C
Copy link
Contributor Author

Hash2C commented Feb 1, 2020

🎉🎉🎉
thanks @ojeytonwilliams

@Hash2C Hash2C deleted the docs/fix-commands-docker-build-locally branch February 1, 2020 13:24
abbathaw pushed a commit to abbathaw/freeCodeCamp that referenced this pull request Jul 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants