Skip to content

Commit

Permalink
Merge pull request #359 from rodrigoprimo/changelog-improvements-3-5-2
Browse files Browse the repository at this point in the history
Add links to all issues in the 3.5.3 and 3.5.2 sections of the changelog
  • Loading branch information
jrfnl authored Feb 23, 2024
2 parents 54f81f2 + 3ae055e commit 1d256d1
Showing 1 changed file with 35 additions and 18 deletions.
53 changes: 35 additions & 18 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -1117,7 +1117,7 @@ Additionally, thanks to [Alexander Turek][@derrabus] for consulting on the repo
- The double arrow becomes the scope opener, and uses a new T_FN_ARROW token type
- The token after the statement (normally a semicolon) becomes the scope closer
- The token is also associated with the opening and closing parenthesis of the statement
- Any functions named "fn" will cause have a T_FN token for the function name, but have no scope information
- Any functions named "fn" will have a T_FN token for the function name, but have no scope information
- Thanks to [Michał Bundyra][@michalbundyra] for the help with this change
- PHP 7.4 numeric separators are now tokenized in the same way when using older PHP versions
- Previously, a number like 1_000 would tokenize as T_LNUMBER (1), T_STRING (_000)
Expand Down Expand Up @@ -1153,24 +1153,34 @@ Additionally, thanks to [Alexander Turek][@derrabus] for consulting on the repo
- Thanks to [Michał Bundyra][@michalbundyra] for the patch

### Fixed
- Fixed bug #2586 : Generic.WhiteSpace.ScopeIndent false positives when indenting open tags at a non tab-stop
- Fixed bug #2638 : Squiz.CSS.DuplicateClassDefinitionSniff sees comments as part of the class name
- Fixed bug [#2586] : Generic.WhiteSpace.ScopeIndent false positives when indenting open tags at a non tab-stop
- Fixed bug [#2638] : Squiz.CSS.DuplicateClassDefinitionSniff sees comments as part of the class name
- Thanks to [Raphael Horber][@rhorber] for the patch
- Fixed bug #2640 : Squiz.WhiteSpace.OperatorSpacing false positives for some negation operators
- Fixed bug [#2640] : Squiz.WhiteSpace.OperatorSpacing false positives for some negation operators
- Thanks to [Jakub Chábek][@grongor] and [Juliette Reinders Folmer][@jrfnl] for the patch
- Fixed bug #2674 : Squiz.Functions.FunctionDeclarationArgumentSpacing prints wrong argument name in error message
- Fixed bug #2676 : PSR12.Files.FileHeader locks up when file ends with multiple inline comments
- Fixed bug #2678 : PSR12.Classes.AnonClassDeclaration incorrectly enforcing that closing brace be on a line by itself
- Fixed bug #2685 : File::getMethodParameters() setting typeHintEndToken for vars with no type hint
- Fixed bug [#2674] : Squiz.Functions.FunctionDeclarationArgumentSpacing prints wrong argument name in error message
- Fixed bug [#2676] : PSR12.Files.FileHeader locks up when file ends with multiple inline comments
- Fixed bug [#2678] : PSR12.Classes.AnonClassDeclaration incorrectly enforcing that closing brace be on a line by itself
- Fixed bug [#2685] : File::getMethodParameters() setting typeHintEndToken for vars with no type hint
- Thanks to [Juliette Reinders Folmer][@jrfnl] for the patch
- Fixed bug #2694 : AbstractArraySniff produces invalid indices when using ternary operator
- Fixed bug [#2694] : AbstractArraySniff produces invalid indices when using ternary operator
- Thanks to [Michał Bundyra][@michalbundyra] for the patch
- Fixed bug #2702 : Generic.WhiteSpace.ScopeIndent false positive when using ternary operator with short arrays
- Fixed bug [#2702] : Generic.WhiteSpace.ScopeIndent false positive when using ternary operator with short arrays

[#2586]: https://github.com/squizlabs/PHP_CodeSniffer/issues/2586
[#2638]: https://github.com/squizlabs/PHP_CodeSniffer/issues/2638
[#2640]: https://github.com/squizlabs/PHP_CodeSniffer/pull/2640
[#2674]: https://github.com/squizlabs/PHP_CodeSniffer/issues/2674
[#2676]: https://github.com/squizlabs/PHP_CodeSniffer/issues/2676
[#2678]: https://github.com/squizlabs/PHP_CodeSniffer/issues/2678
[#2685]: https://github.com/squizlabs/PHP_CodeSniffer/pull/2685
[#2694]: https://github.com/squizlabs/PHP_CodeSniffer/issues/2694
[#2702]: https://github.com/squizlabs/PHP_CodeSniffer/issues/2702

## [3.5.2] - 2019-10-28
### Changed
- Generic.ControlStructures.DisallowYodaConditions now returns less false positives
- False positives were being returned for array comparisions, or when performing some function calls
- False positives were being returned for array comparisons, or when performing some function calls
- Squiz.WhiteSpace.SemicolonSpacing.Incorrect error message now escapes newlines and tabs
- Provides a clearer error message as whitespace is now visible
- Also allows for better output for report types such as CSV and XML
Expand All @@ -1179,13 +1189,20 @@ Additionally, thanks to [Alexander Turek][@derrabus] for consulting on the repo
- Thanks to [Craig Duncan][@duncan3dc] for the patch

### Fixed
- Fixed bug #2654 : Incorrect indentation for arguments of multiline function calls
- Fixed bug #2656 : Squiz.WhiteSpace.MemberVarSpacing removes comments before first member var during auto fixing
- Thanks to [Juliette Reinders Folmer][@jrfnl] for the patch
- Fixed bug #2663 : Generic.NamingConventions.ConstructorName complains about old constructor in interfaces
- Fixed bug #2664 : PSR12.Files.OpenTag incorrectly identifies PHP file with only an opening tag
- Fixed bug #2665 : PSR12.Files.ImportStatement should not apply to traits
- Fixed bug #2673 : PSR12.Traits.UseDeclaration does not allow comments or blank lines between use statements
- Fixed bug [#2654] : Incorrect indentation for arguments of multiline function calls
- Fixed bug [#2656] : Squiz.WhiteSpace.MemberVarSpacing removes comments before first member var during auto fixing
- Thanks to [Juliette Reinders Folmer][@jrfnl] for the patch
- Fixed bug [#2663] : Generic.NamingConventions.ConstructorName complains about old constructor in interfaces
- Fixed bug [#2664] : PSR12.Files.OpenTag incorrectly identifies PHP file with only an opening tag
- Fixed bug [#2665] : PSR12.Files.ImportStatement should not apply to traits
- Fixed bug [#2673] : PSR12.Traits.UseDeclaration does not allow comments or blank lines between use statements

[#2654]: https://github.com/squizlabs/PHP_CodeSniffer/issues/2654
[#2656]: https://github.com/squizlabs/PHP_CodeSniffer/pull/2656
[#2663]: https://github.com/squizlabs/PHP_CodeSniffer/issues/2663
[#2664]: https://github.com/squizlabs/PHP_CodeSniffer/issues/2664
[#2665]: https://github.com/squizlabs/PHP_CodeSniffer/issues/2665
[#2673]: https://github.com/squizlabs/PHP_CodeSniffer/issues/2673

## [3.5.1] - 2019-10-17
### Changed
Expand Down

0 comments on commit 1d256d1

Please sign in to comment.