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
magento/magento2#24043: Better exception handling during cli commands. #24734
magento/magento2#24043: Better exception handling during cli commands. #24734
Conversation
Hi @p-bystritsky. Thank you for your contribution
For more details, please, review the Magento Contributor Guide documentation. |
Hi @sidolov, thank you for the review.
|
@p-bystritsky : out of curiosity, will this also fix #24678 ? |
I can confirm that applying this PR fixes #24678, nice! @p-bystritsky: can you edit your post and add #24678 to the fixed issues list so it will get automatically marked as fixed when this PR gets merged? Thanks! |
I do have one remark about only logging the trace to the |
@hostep agree with you, @p-bystritsky could you take a look on it? |
cf0e68b
to
a56699c
Compare
a56699c
to
dfd481d
Compare
LGTM! Thanks! |
Hi @sidolov, thank you for the review. |
✔️ QA passed |
Hi @p-bystritsky, thank you for your contribution! |
Description (*)
This PR is updating https://github.com/magento/magento2ce/commit/9ffe7eb2d3e6fcbf18eb39b6b847e99b0550a028
Fixed Issues (if relevant)
Manual testing scenarios (*)
Questions or comments
Contribution checklist (*)