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

Simplified the contribution article for Symfony Docs #6367

Closed
wants to merge 7 commits into from

Conversation

javiereguiluz
Copy link
Member

After last Saturday's Hack Day, I realized that we could simplify the documentation of the contribution process significantly. I'll propose changes to the code contribution, but let's start with the doc contribution.

One of the essential principles of the Symfony project is that **documentation
is as important as code**. That's why a great amount of resources are dedicated
to documenting new features and to keeping the rest of the documentation up-to-
date.
Copy link
Member

Choose a reason for hiding this comment

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

"up-to-date" must be on one line. Otherwise, the newline is treated as a space.


$ git remote -v
origin https://github.com/<YOUR GITHUB USERNAME>/symfony-docs.git (fetch)
origin https://github.com/<YOUR GITHUB USERNAME>/symfony-docs.git (push)
Copy link
Member

Choose a reason for hiding this comment

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

For your own repository, their shouldn't be HTTPS URLs (we cloned using Git over SSH before).

@javiereguiluz
Copy link
Member Author

@xabbuh I've made all the changes you asked for. Thanks for the review. This is now ready for the final review.

automatically and you can then create the Pull Request.

If your contribution is large or if you prefer to work on your own computer,
keep reading this guide to learn an alternative way to send pull requests to
Copy link
Member

Choose a reason for hiding this comment

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

to the

@xabbuh
Copy link
Member

xabbuh commented May 21, 2016

👍

@javiereguiluz
Copy link
Member Author

I've fixed the last errors and suggestions. Thanks!

* Symfony documentation is published under a
:doc:`Creative Commons BY-SA 3.0 License </contributing/documentation/license>`
and all your contributions will implicitly adhere to that license.

.. _minor-changes-e-g-typos
Copy link
Member

Choose a reason for hiding this comment

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

missing : at the end

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed. Thanks.

First, switch to the oldest maintained Symfony branch which contains the error,
which is usually ``2.3``. Then, click on the **edit this page** button on the
upper right corner. Now edit the content and describe your changes within the
GitHub frontend.
Copy link
Member

Choose a reason for hiding this comment

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

I think we should remove this. If it's a small change, this is a big step to figure out, and in about 1 month. 2.3 won't be the oldest version anymore :). Also, it shouldn't be a problem in most cases to merge to the correct branch.

Copy link
Member

Choose a reason for hiding this comment

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

To be clear, I mean remove the first sentence only (then re-word the second)

A picture here would be sweet too.

you are probably comfortable with all the Git-magic involved in the process.
That's why your next contributions would be much faster. Here you can find the
complete steps to contribute to the Symfony documentation, which you can use as
a **checklist**:
Copy link
Member

Choose a reason for hiding this comment

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

Check you out! You've made your first contribution to the Symfony documentation!
Somebody throw a party! Your first contribution took a little extra time because you
needed to learn a few standards and setup your computer. But now, you're dangerous.

Here is a complete checklist of steps that will guide you through your next contribution
to the Symfony docs:

@weaverryan
Copy link
Member

Sorry - late to the party. I suggested some changes - do with them what you want @javiereguiluz - some of them are just Ryan-isms 💃

👍

@javiereguiluz
Copy link
Member Author

@weaverryan thanks for your great review. I've fixed everything and as a bonus I've added some screenshots too.

@xabbuh
Copy link
Member

xabbuh commented May 21, 2016

Thank you Javier.

@xabbuh xabbuh closed this May 21, 2016
xabbuh added a commit that referenced this pull request May 21, 2016
…iereguiluz)

This PR was squashed before being merged into the 2.3 branch (closes #6367).

Discussion
----------

Simplified the contribution article for Symfony Docs

After last Saturday's Hack Day, I realized that we could simplify the documentation of the contribution process significantly. I'll propose changes to the code contribution, but let's start with the doc contribution.

Commits
-------

eeb6abb Simplified the contribution article for Symfony Docs
@weaverryan
Copy link
Member

Yes, awesome!

@javiereguiluz javiereguiluz deleted the simplify_docs branch May 24, 2018 16:04
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.

None yet

5 participants