Skip to content

Conversation

rmsundar1
Copy link
Contributor

Description (*)

  • Fixed - False positive - Direct throw of generic Exception is discouraged

Fixed Issues (if relevant)

  1. Fixes False positive - Direct throw of generic Exception is discouraged #194
Copy link
Collaborator

@ihor-sviziev ihor-sviziev left a comment

Choose a reason for hiding this comment

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

Please do 2 small fixes, and then I'll approve your changes :)

return [
20 => 1
];

Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed


}
return [];

Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

ihor-sviziev
ihor-sviziev previously approved these changes Nov 30, 2021
rmsundar1 and others added 2 commits December 15, 2021 18:31
Co-authored-by: Sergio Vera <sergio.vera@gmail.com>
@rmsundar1 rmsundar1 requested a review from svera December 15, 2021 13:10
@ihor-sviziev ihor-sviziev removed their assignment Dec 16, 2021
@svera
Copy link
Contributor

svera commented Dec 16, 2021

@magento import pr to magento-commerce/magento-coding-standard

@m2-github-services
Copy link
Contributor

@svera the Pull Request is successfully imported.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants