Skip to content

Conversation

ogizanagi
Copy link
Contributor

Q A
Branch? 2.7
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets N/A
License MIT
Doc PR N/A

I think it's a good idea to reference this guide on the repository main page and contributing guide. Especially as it explains processes for reviewing that are specific to Symfony (Carson Bot).

@xabbuh
Copy link
Member

xabbuh commented Dec 19, 2016

👍

@javiereguiluz
Copy link
Member

👍 a very nice addition!

@fabpot
Copy link
Member

fabpot commented Dec 19, 2016

Thank you @ogizanagi.

@fabpot fabpot merged commit 313defa into symfony:2.7 Dec 19, 2016
fabpot added a commit that referenced this pull request Dec 19, 2016
This PR was merged into the 2.7 branch.

Discussion
----------

Mention the community review guide

| Q             | A
| ------------- | ---
| Branch?       | 2.7
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | N/A
| License       | MIT
| Doc PR        | N/A

I think it's a good idea to reference this guide on the repository main page and contributing guide. Especially as it explains processes for reviewing that are specific to Symfony (Carson Bot).

Commits
-------

313defa Mention the community review guide
@ogizanagi ogizanagi deleted the community_reviews branch December 19, 2016 17:27
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.

6 participants