Skip to content

Describe how to get a stack trace with Symfony #13699

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
Sep 20, 2020

Conversation

greg0ire
Copy link
Contributor

As a maintainer, I often struggle to get stack traces in bug reports. I
miss a good resource to describe how to do it with Symfony. This aims at
showing developers how to do that, but also to help them understand more
about stack traces.

This PR does not fix a bug, nor does it document a new feature, so I'm not sure which branch I should target here.

@greg0ire
Copy link
Contributor Author

@OskarStark can you please help DOCtor-RST? I think there is an issue with https://github.com/OskarStark/doctor-rst/blob/e4b46021e110dc5aed0ee62d6513a61ab9ebf64f/src/Rule/PhpPrefixBeforeBinConsole.php , it shouldn't report


contributing/code/stack_trace.rst ✘
  161: Please add "php" prefix before "bin/console"
   ->  Symfony\Component\Console\Application->run() at /home/greg/dev/sf-demo/bin/console:42

@greg0ire
Copy link
Contributor Author

greg0ire commented Jun 6, 2020

I made a PR to doctor-RST: OskarStark/doctor-rst#697

@greg0ire greg0ire closed this Jun 8, 2020
@greg0ire greg0ire reopened this Jun 8, 2020
@greg0ire
Copy link
Contributor Author

greg0ire commented Jun 8, 2020

Aaaaand fixed :)

Copy link
Contributor

@faizanakram99 faizanakram99 left a comment

Choose a reason for hiding this comment

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

I think it's great, thank you

Copy link
Contributor

@noniagriconomie noniagriconomie left a comment

Choose a reason for hiding this comment

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

Some final reading from my side
Thanks @greg0ire

@greg0ire greg0ire force-pushed the getting-a-stack-trace branch from f2e4f57 to 65f1e89 Compare June 25, 2020 19:18
@greg0ire
Copy link
Contributor Author

greg0ire commented Sep 3, 2020

@noniagriconomie I noticed today that I had local changes to address your comments that I had not commited. Please review again and mark discussions as resolved if they are indeed resolved.

Copy link
Contributor

@noniagriconomie noniagriconomie left a comment

Choose a reason for hiding this comment

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

@greg0ire ok for me :)

Copy link
Contributor

@OskarStark OskarStark left a comment

Choose a reason for hiding this comment

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

Only some minor comments.

@symfony/team-symfony-docs I am 👍🏻 for merging it. It brings additional understanding and help for new developers.

Thank you for the time to write this down @greg0ire🙏

@greg0ire greg0ire force-pushed the getting-a-stack-trace branch 2 times, most recently from f3a10cc to 9f42830 Compare September 20, 2020 08:41
As a maintainer, I often struggle to get stack traces in bug reports. I
miss a good resource to describe how to do it with Symfony. This aims at
showing developers how to do that, but also to help them understand more
about stack traces.
@xabbuh xabbuh changed the base branch from master to 4.4 September 20, 2020 09:54
@xabbuh xabbuh requested a review from HeahDude as a code owner September 20, 2020 09:55
@xabbuh xabbuh force-pushed the getting-a-stack-trace branch from 9f42830 to d0a97ae Compare September 20, 2020 09:55
@xabbuh
Copy link
Member

xabbuh commented Sep 20, 2020

Thank you @greg0ire.

@xabbuh xabbuh merged commit 90fc583 into symfony:4.4 Sep 20, 2020
@greg0ire greg0ire deleted the getting-a-stack-trace branch September 20, 2020 10:00
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