-
Notifications
You must be signed in to change notification settings - Fork 942
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
Update getting started docs #3871
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly good! A few criticisms.
docs/development/getting-started.rst
Outdated
|
||
Warehouse needs access to a few local ports in order to run, namely ports | ||
``80``, ``5433``, and ``9000``. You can check if a port is in use by running | ||
the following command (in this example, checking port ``80``: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have a missing )
here.
docs/development/getting-started.rst
Outdated
the web container is listening on port 80. It's accessible at | ||
``http://localhost:80/``. | ||
This means that all the services are up, and web container is listening on port | ||
80. It's accessible at ``http://localhost:80/``. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest we make http://localhost:80/
a clickable link.
docs/development/getting-started.rst
Outdated
|
||
If the port is in use, the command will produce output, and you will need to | ||
determine what is occupying the port and shut down the service. Otherwise, the | ||
port is available for Warehouse to use, and you can continue. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think either this should be in the troubleshooting section, or we should phrase this instruction more imperatively:
Check each of those three port numbers with
lsof
.
@brainwane Addressed your feedback. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
This fixes some of the issues listed at https://public.etherpad-mozilla.org/p/issues-from-sprint, namely:
make serve
commandmake initdb
commandCloses #2673, closes #3861.