Skip to content

Revamped the documentation about "Contributing Docs" #4223

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

Merged
merged 10 commits into from
Sep 16, 2014
Merged

Revamped the documentation about "Contributing Docs" #4223

merged 10 commits into from
Sep 16, 2014

Conversation

javiereguiluz
Copy link
Member

Q A
Doc fix? yes
New docs? yes
Applies to 2.3+
Fixed tickets -

This is the first step into completely revamping the Contributing section documentation. The main changes are:

  • A much more friendly and welcoming tone.
  • Thorough and step-by-step explanations that assumes no previous knowledge from the reader.
  • The use of real examples to illustrate the explanations, instead of using abstract concepts.
  • The old Managing Releases section has been completely removed. This has nothing to do with contributing documentations. It's just documentation for a process that only concerns to doc maintainers.

@stof
Copy link
Member

stof commented Sep 13, 2014

the description of the PR seems incomplete

@javiereguiluz
Copy link
Member Author

@stof it's fixed now :)

.. code-block:: text

https://github.com/symfony/symfony-docs
* Symfony documentation is written using `reStructuredText`_ markup language.
Copy link
Member

Choose a reason for hiding this comment

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

There's no target given for "reStructuredText" at the end of the file. Should probably be the same as in format.rst.

Copy link
Member

Choose a reason for hiding this comment

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

Never mind, it seems that you fixed it in the meantime. :)

| Fixed tickets | #1075
| New docs? | no
| Applies to | 2.3+
| Fixed tickets | #10575
Copy link
Member

Choose a reason for hiding this comment

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

why changing this number ? The highest number in the doc repo is still far from 10000

Copy link
Member Author

Choose a reason for hiding this comment

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

I wanted to use a real looking but non-existent number.

Copy link
Member

Choose a reason for hiding this comment

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

Why shouldn't it exists? I find the previous better

@stof
Copy link
Member

stof commented Sep 13, 2014

btw, the rule about relative links should be added in the doc standards. It is not written currently

@javiereguiluz
Copy link
Member Author

The rule about internal relative links has been added here: 991ff6f

directive to show the configuration in all supported configuration formats
(``PHP``, ``YAML``, and ``XML``)
(``PHP``, ``YAML``, and ``XML``). Example:
Copy link
Member

Choose a reason for hiding this comment

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

you should remove the serial comma and maybe link to the standards here to show the full list of languages?


$ git clone git://github.com/YOURUSERNAME/symfony-docs.git
Let's imagine that you want to improve the installation chapter of the Symfony
book. In order to do those changes, follow these steps:
Copy link
Member

Choose a reason for hiding this comment

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

In order to make your changes,

@javiereguiluz
Copy link
Member Author

Ryan, thanks for your great review. I'm really sorry for having made so many mistakes :(

@weaverryan
Copy link
Member

Pff, you only make as many mistakes as I do :) - it's much easier to do "cleanup" on someone else's work. Thanks very much for this!

@weaverryan weaverryan merged commit 88ddbb1 into symfony:2.3 Sep 16, 2014
weaverryan added a commit that referenced this pull request Sep 16, 2014
…aviereguiluz)

This PR was merged into the 2.3 branch.

Discussion
----------

Revamped the documentation about "Contributing Docs"

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | yes
| Applies to    | 2.3+
| Fixed tickets | -

This is the first step into completely revamping the *Contributing* section documentation. The main changes are:

  * A much more friendly and welcoming tone.
  * Thorough and step-by-step explanations that assumes no previous knowledge from the reader.
  * The use of real examples to illustrate the explanations, instead of using abstract concepts.
  * The old [Managing Releases](http://symfony.com/doc/current/contributing/documentation/overview.html#managing-releases) section has been completely removed. This has nothing to do with contributing documentations. It's just documentation for a process that only concerns to doc maintainers.

Commits
-------

88ddbb1 Fixed all the errors found by Ryan
baa06b5 Removed two highlight formats which are "experimental" and not used by end users
9bcd33d More and more fixes and improvements
15f3c94 Added another bunch of fixes suggested by reviewers
ff66e94 Added lots of fixes suggested by reviewers
991ff6f Added a note about not using relative internal links in the doc
79375ae Switched another relative link into an absolute reference
4751eb9 lways use absolute links instead of relative for internal doc links
53c3a16 Added missing link
723603c Revamped the documentation about "Contributing Docs"
weaverryan added a commit that referenced this pull request Oct 2, 2014
This PR was merged into the 2.3 branch.

Discussion
----------

[Contributing] tweaks to the contribution chapter

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | all
| Fixed tickets |

Some tweaks for the changes made in #4223.

@javiereguiluz Sorry,  didn't find the time to review it in the first run.

Commits
-------

f4b0b8e tweaks to the contribution chapter
@wouterj wouterj mentioned this pull request Jan 5, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants