-
Notifications
You must be signed in to change notification settings - Fork 76
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
Clarify the need to setup Wordpress locally #124
Comments
Well, that and:
This comes directly from the setup instructions. |
Sounds like there are definitely some things we can clarify on the site. Assigning this one to me, I'll work on it.
👍 |
OK, apparently I'm blind. I'll leave it to you to decide if it should be highlighted more visibly or if current instructions are fine. |
I think the README can certainly use some better formatting/editing so that there's not a massive block of expected output before the next (important) step. |
Yeah, that is most likely what made me jump too far into the list. |
I wanted to fix one thing on jquery.com and I didn't do anything there for a long time so I decided to try with freshly cloned repos with the new Vagrant setup. I started from:
https://github.com/jquery/jquery.com/blob/master/README.md
which redirects to:
https://github.com/jquery/jquery-wp-content
(basically just "install Virtualbox & Vagrant & run
vagrant up
" - simple) and:http://contribute.jquery.org/web-sites/#workflow
However, it didn't work for me, I was getting
wordpress-validate
errors about and unknown error that may mean Wordpress or the database is not set properly. Now, instructions at the contribute url say:which may suggest you first need to set up the Wordpress instance... but with having most of the setup handled by Vagrant, I thought that's handled as well.
It would save me (& maybe more people) some time if a step saying explicitly about going to http://vagrant.jquery.com & setting it up first was introduced.
The text was updated successfully, but these errors were encountered: