bug: TrailingCommaInMultilineFixer - do not add trailing comma when there is no break line after last element

[kubawerlos]

Fixes #6675

[julienfalque]

Looks good to me but since we actually had test cases for adding the comma, should we consider this a BC break and introduce a new option?

[coveralls]

Pull Request Test Coverage Report for Build 3388318456

• 4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
• 1 unchanged line in 1 file lost coverage.
• Overall coverage decreased (-0.004%) to 92.917%

---

Files with Coverage Reduction	New Missed Lines	%
src/Fixer/ControlStructure/TrailingCommaInMultilineFixer.php	1	94.94%

Totals
Change from base Build 3365138575:	-0.004%
Covered Lines:	17276
Relevant Lines:	18593

---

💛 - Coveralls

[VincentLanglet]

Looks good to me but since we actually had test cases for adding the comma, should we consider this a BC break and introduce a new option?

It was introduced in #4238 without anybody talking about this special test cases. So technically we can consider this as a BC break but I really feel like it's more an improvement. Having a special option for this seems too much. I don't expect any complaint "I want the code like it was before"

[julienfalque]

Thank you @kubawerlos.