[WIP] The Testing Documentation section is unclear.

[clemens-tolboom]

I installed Sphinx then did the git submodules next generated the docs thus skipping the bash install.sh

* Install `Sphinx`_;
* Install the Sphinx extensions using git submodules: ``git submodule update --init``;
* (Optionally) Install the bundle docs and CMF docs: ``bash install.sh``;
* Run ``make html`` and view the generated HTML in the ``build`` directory.

It is unclear what to expect after each step.

Opening ie $ open _build/html/cookbook/controller/error_pages.html it does not look at all similar to http://symfony.com/doc/current/cookbook/controller/error_pages.html

So I ran bash install.sh expecting that should do it. Nope.

I would propose the following changes:

• Add text about test sphinx is already installed.
• We should explain how the build will look like.
• We should explain why bash install.sh is optional.
• Tell about the additional _ext directory and how not to add it to the PR.

[wouterj]

Symfony uses a custom Symfony theme, you'll get the default Sphinx theme. That's why it's different.

Tell about the additional _ext directory and how not to add it to the PR.

If everything is correct, the gitignore should ignore this _exts directory: https://github.com/symfony/symfony-docs/blob/master/.gitignore#L8

---

I don't think we should put much time into this. We are working hard to facilitate a demo for each PR automatically (in the same way as Travis currently works here). When we do that, we no longer need any of this.

[clemens-tolboom]

I disagree with

When we do that, we no longer need any of this.

as it explains how the proces works thus learning something new. /me developer

[wouterj]

@javiereguiluz can you maybe include these suggestions to your PR about revamping the contributing guide? Thanks!

[xabbuh]

@wouterj #4223 is already merged.

[wouterj]

Pff, I'm happy to have you around, I can't keep track of everything... :p

In that case, we have to update this PR. @clemens-tolboom do you have any suggestions?

[clemens-tolboom]

#4223 does not address this PR. So it still stands. Not sure I find the time as this issue came to life due to other doc PRs. I know what to expect next time. But it may help others.

[wouterj]

Hi @clemens-tolboom!

I'm going to close this PR as it doesn't add that much information at the moment and you said you didn't know if you had any time to update this PR. I've created an issue with the tasks you mentioned in the start post of this PR: #5259 (please note that some are no longer relevant).

Thanks for bringing this up!

[clemens-tolboom]

Thanks for the update