Skip to content
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

[ci skip] Clean up documentation merged into main repo #4621

Merged
merged 1 commit into from
Mar 25, 2016

Conversation

pjedrzejewski
Copy link
Member

Q A
Related tickets #4581
License MIT

@pjedrzejewski pjedrzejewski added the Documentation Documentation related issues and PRs - requests, fixes, proposals. label Mar 25, 2016
@michalmarcinkowski michalmarcinkowski merged commit c78447f into Sylius:master Mar 25, 2016
@michalmarcinkowski
Copy link
Contributor

👍

@@ -31,7 +31,7 @@ Create a dedicated branch for your changes (for organization):

.. code-block:: bash

$ git checkout -b improving_foo_and_bar
$ git checkout -b docs/improving_foo_and_bar
Copy link
Contributor

Choose a reason for hiding this comment

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

Just a note, I think using dashes is more popular than underscores.

Copy link
Member Author

Choose a reason for hiding this comment

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

This is copied from Symfony docs, I wanted to leave it intact for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Documentation related issues and PRs - requests, fixes, proposals.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants