Resolve incompatibility between standards

[fredden]

This will fix #6 while we wait for the Magento2 standard to adopt PSR-12.

[leonhelmus]

looks good to me!

[Anve94]

I'm not sure if we still need to exclude the FirstExpressionLine if multi-line if-statements are formatted following PSR-12 regardless, e.g.

// PSR-12:
// Expressions in parentheses MAY be split across multiple lines, where each subsequent line is indented at least once.
// When doing so, the first condition MUST be on the next line.

if (
    $firstExpression ||
    $secondExpression ||
    $thirdExpression
) {
    // Do something
}

But this is not blocking for me.

[fredden]

As per #6, we have to two inherited rulesets in YouweMagento2, which conflict with each other.

• Youwe pulls in PSR12.ControlStructures.ControlStructureSpacing
• Magento2 pulls in PSR2.ControlStructures.ControlStructureSpacing

This pull request removes the PSR-2 rule; the PSR-12 rule is still in this standard.

References:

1. https://github.com/YouweGit/coding-standard-magento2/blob/2.1.0/src/YouweMagento2/ruleset.xml#L16 -> https://github.com/YouweGit/coding-standard/blob/3.5.1/src/Youwe/ruleset.xml#L11 -> https://github.com/YouweGit/coding-standard/blob/3.5.1/src/GlobalCommon/ruleset.xml#L16
2. https://github.com/YouweGit/coding-standard-magento2/blob/2.1.0/src/YouweMagento2/ruleset.xml#L33 -> https://github.com/magento/magento-coding-standard/blob/v31/Magento2/ruleset.xml#L582
3. Upgrade PSR2 -> PSR12 for ControlStructureSpacing magento/magento-coding-standard#416