Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Contributing][Code] add note on Symfony SE forks for bug reports #4072

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 1 commit into from
Aug 13, 2014

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Jul 30, 2014

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

As @javiereguiluz stated in symfony/symfony#11494, the way to report bugs can be improved. One of the things that could be done, is to add a note to make a bug reproducable by forking the Symfony Standard Edition with the modifications needed to reproduce a particular issue.

@xabbuh xabbuh changed the title add note on Symfony SE forks for bug reports [Contributing][Code] add note on Symfony SE forks for bug reports Jul 30, 2014

* *(optional)* It helps others a lot to fix the bug you discovered if you
fork the `Symfony Standard Edition`_ and reproduce your issue on a new
branch.
Copy link
Member

Choose a reason for hiding this comment

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

Be careful here. Forking the Symfony SE is not the right for all bug reports. When reporting a bug on some standalone component, providing a failing test case if much better. Forking should be used only for problems spanning multiple layers.

Copy link
Member Author

Choose a reason for hiding this comment

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

You're right. Maybe we should better intodruce a subsection explaining both ways to make a bug reproducable.

@xabbuh
Copy link
Member Author

xabbuh commented Aug 1, 2014

I reworded the item a bit and moved it just below the one that advises to provide a unit test. I think it's more clear this way.

@weaverryan
Copy link
Member

Nice small edition 👍 and thanks!

@weaverryan weaverryan merged commit ebf1415 into symfony:2.3 Aug 13, 2014
weaverryan added a commit that referenced this pull request Aug 13, 2014
… reports (xabbuh)

This PR was merged into the 2.3 branch.

Discussion
----------

[Contributing][Code] add note on Symfony SE forks for bug reports

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

As @javiereguiluz stated in symfony/symfony#11494, the way to report bugs can be improved. One of the things that could be done, is to add a note to make a bug reproducable by forking the Symfony Standard Edition with the modifications needed to reproduce a particular issue.

Commits
-------

ebf1415 add note on Symfony SE forks for bug reports
@xabbuh xabbuh deleted the symfony-11494 branch August 13, 2014 21:27
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.

3 participants