Skip to content

Conversation

@calumbrodie
Copy link
Contributor

No description provided.

Copy link
Contributor

Choose a reason for hiding this comment

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

can you break this on multiple lines please?

and can you add a similar note to the one ddboer did on the proxy doc, that you should not use this for headers that are always the same.

@calumbrodie
Copy link
Contributor Author

@stof Thanks (I did wonder what the failed build was about). Is there a quicker way to validate the documentation other than attempting to build it with sphinx (for next time)?

@stof
Copy link
Member

stof commented Aug 13, 2014

@calumbrodie not that I know of (which is why our Travis build is building it)

Copy link
Member

Choose a reason for hiding this comment

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

Please don't remove the empty line between the reference and the title

@dbu
Copy link
Contributor

dbu commented Aug 14, 2014

@stof https://github.com/stof Thanks (I did wonder what the failed
build was about). Is there a quicker way to validate the documentation
other than attempting to build it with sphinx (for next time)?

if you just do the occasional fix, travis is fine. when starting to
write whole doc sections, i just installed sphinx - its not that hard,
and once you have it running, its very easy to build locally. i
installed it according to http://sphinx-doc.org/ and then its just make html.

@ddeboer ddeboer merged commit 4836698 into FriendsOfSymfony:master Aug 14, 2014
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.

4 participants