Skip to content

Commit 2db63c7

Browse files
authored
Merge pull request #401 from rodrigoprimo/changelog-improvements-2-5-0
Add links to all issues in the 2.5.1 and 2.5.0 sections of the changelog
2 parents 65d36da + 47f6566 commit 2db63c7

File tree

1 file changed

+78
-38
lines changed

1 file changed

+78
-38
lines changed

CHANGELOG.md

Lines changed: 78 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -3352,51 +3352,77 @@ Additionally, thanks to [Alexander Turek][@derrabus] for consulting on the repo
33523352
- Squiz ComparisonOperatorUsageSniff no longer hangs on JS FOR loops that don't use semicolons
33533353
- PHP_CodesSniffer now includes the composer autoload.php file, if there is one
33543354
- Thanks to [Klaus Purer][@klausi] for the patch
3355-
- Added error Squiz.Commenting.FunctionComment.ScalarTypeHintMissing for PHP7 only (request #858)
3355+
- Added error Squiz.Commenting.FunctionComment.ScalarTypeHintMissing for PHP7 only (request [#858])
33563356
- These errors were previously reported as Squiz.Commenting.FunctionComment.TypeHintMissing on PHP7
33573357
- Disable this error message in a ruleset.xml file if your code needs to run on both PHP5 and PHP7
33583358
- The PHP 5.6 __debugInfo magic method no longer produces naming convention errors
33593359
- Thanks to [Michael Nowack][@syranez] for the patch
3360-
- PEAR and Squiz FunctionComment sniffs now support variadic functions (request #841)
3360+
- PEAR and Squiz FunctionComment sniffs now support variadic functions (request [#841])
33613361

33623362
### Fixed
3363-
- Fixed bug #622 : Wrong detection of Squiz.CSS.DuplicateStyleDefinition with media queries
3364-
- Fixed bug #752 : The missing exception error is reported in first found DocBlock
3365-
- Fixed bug #794 : PSR2 MultiLineFunctionDeclaration forbids comments after opening parenthesis of a multiline call
3366-
- Fixed bug #820 : PEAR/PSR2 FunctionCallSignature sniffs suggest wrong indent when there are multiple arguments on a line
3367-
- Fixed bug #822 : Ruleset hard-coded file paths are not used if not running from the same directory as the ruleset
3368-
- Fixed bug #825 : FunctionCallArgumentSpacing sniff complains about more than one space before comment in multi-line function call
3369-
- Fixed bug #828 : Null classname is tokenized as T_NULL instead of T_STRING
3370-
- Fixed bug #829 : Short array argument not fixed correctly when multiple function arguments are on the same line
3371-
- Fixed bug #831 : PHPCS freezes in an infinite loop under Windows if no standard is passed
3372-
- Fixed bug #832 : Tokenizer does not support context sensitive parsing
3363+
- Fixed bug [#622] : Wrong detection of Squiz.CSS.DuplicateStyleDefinition with media queries
3364+
- Fixed bug [#752] : The missing exception error is reported in first found DocBlock
3365+
- Fixed bug [#794] : PSR2 MultiLineFunctionDeclaration forbids comments after opening parenthesis of a multiline call
3366+
- Fixed bug [#820] : PEAR/PSR2 FunctionCallSignature sniffs suggest wrong indent when there are multiple arguments on a line
3367+
- Fixed bug [#822] : Ruleset hard-coded file paths are not used if not running from the same directory as the ruleset
3368+
- Fixed bug [#825] : FunctionCallArgumentSpacing sniff complains about more than one space before comment in multi-line function call
3369+
- Fixed bug [#828] : Null classname is tokenized as T_NULL instead of T_STRING
3370+
- Fixed bug [#829] : Short array argument not fixed correctly when multiple function arguments are on the same line
3371+
- Fixed bug [#831] : PHPCS freezes in an infinite loop under Windows if no standard is passed
3372+
- Fixed bug [#832] : Tokenizer does not support context sensitive parsing
33733373
- Thanks to [Jaroslav Hanslík][@kukulich] for the patch
3374-
- Fixed bug #835 : PEAR.Functions.FunctionCallSignature broken when closure uses return types
3375-
- Fixed bug #838 : CSS indentation fixer changes color codes
3374+
- Fixed bug [#835] : PEAR.Functions.FunctionCallSignature broken when closure uses return types
3375+
- Fixed bug [#838] : CSS indentation fixer changes color codes
33763376
- Thanks to [Klaus Purer][@klausi] for the patch
3377-
- Fixed bug #839 : "__()" method is marked as not camel caps
3377+
- Fixed bug [#839] : "__()" method is marked as not camel caps
33783378
- Thanks to [Tim Bezhashvyly][@tim-bezhashvyly] for the patch
3379-
- Fixed bug #852 : Generic.Commenting.DocComment not finding errors when long description is omitted
3380-
- Fixed bug #854 : Return typehints in interfaces are not reported as T_RETURN_TYPE
3379+
- Fixed bug [#852] : Generic.Commenting.DocComment not finding errors when long description is omitted
3380+
- Fixed bug [#854] : Return typehints in interfaces are not reported as T_RETURN_TYPE
33813381
- Thanks to [Jaroslav Hanslík][@kukulich] for the patch
3382-
- Fixed bug #855 : Capital letter detection for multibyte strings doesn't work correctly
3383-
- Fixed bug #857 : PSR2.ControlStructure.SwitchDeclaration shouldn't check indent of curly brace closers
3384-
- Fixed bug #859 : Switch statement indention issue when returning function call with closure
3385-
- Fixed bug #861 : Single-line arrays and function calls can generate incorrect indentation errors
3386-
- Fixed bug #867 : Squiz.Strings.DoubleQuoteUsage broken for some escape codes
3382+
- Fixed bug [#855] : Capital letter detection for multibyte strings doesn't work correctly
3383+
- Fixed bug [#857] : PSR2.ControlStructure.SwitchDeclaration shouldn't check indent of curly brace closers
3384+
- Fixed bug [#859] : Switch statement indention issue when returning function call with closure
3385+
- Fixed bug [#861] : Single-line arrays and function calls can generate incorrect indentation errors
3386+
- Fixed bug [#867] : Squiz.Strings.DoubleQuoteUsage broken for some escape codes
33873387
- Thanks to [Jack Blower][@ElvenSpellmaker] for the help with the fix
3388-
- Fixed bug #21005 : Incorrect indent detection when multiple properties are initialized to arrays
3389-
- Fixed bug #21010 : Incorrect missing colon detection in CSS when first style is not on new line
3390-
- Fixed bug #21011 : Incorrect error message text when newline found after opening brace
3388+
- Fixed bug [#21005] : Incorrect indent detection when multiple properties are initialized to arrays
3389+
- Fixed bug [#21010] : Incorrect missing colon detection in CSS when first style is not on new line
3390+
- Fixed bug [#21011] : Incorrect error message text when newline found after opening brace
3391+
3392+
[#622]: https://github.com/squizlabs/PHP_CodeSniffer/issues/622
3393+
[#752]: https://github.com/squizlabs/PHP_CodeSniffer/issues/752
3394+
[#794]: https://github.com/squizlabs/PHP_CodeSniffer/issues/794
3395+
[#820]: https://github.com/squizlabs/PHP_CodeSniffer/issues/820
3396+
[#822]: https://github.com/squizlabs/PHP_CodeSniffer/issues/822
3397+
[#825]: https://github.com/squizlabs/PHP_CodeSniffer/issues/825
3398+
[#828]: https://github.com/squizlabs/PHP_CodeSniffer/issues/828
3399+
[#829]: https://github.com/squizlabs/PHP_CodeSniffer/issues/829
3400+
[#831]: https://github.com/squizlabs/PHP_CodeSniffer/issues/831
3401+
[#832]: https://github.com/squizlabs/PHP_CodeSniffer/issues/832
3402+
[#835]: https://github.com/squizlabs/PHP_CodeSniffer/issues/835
3403+
[#838]: https://github.com/squizlabs/PHP_CodeSniffer/pull/838
3404+
[#839]: https://github.com/squizlabs/PHP_CodeSniffer/issues/839
3405+
[#841]: https://github.com/squizlabs/PHP_CodeSniffer/issues/841
3406+
[#852]: https://github.com/squizlabs/PHP_CodeSniffer/issues/852
3407+
[#854]: https://github.com/squizlabs/PHP_CodeSniffer/issues/854
3408+
[#855]: https://github.com/squizlabs/PHP_CodeSniffer/pull/855
3409+
[#857]: https://github.com/squizlabs/PHP_CodeSniffer/issues/857
3410+
[#858]: https://github.com/squizlabs/PHP_CodeSniffer/issues/858
3411+
[#859]: https://github.com/squizlabs/PHP_CodeSniffer/issues/859
3412+
[#861]: https://github.com/squizlabs/PHP_CodeSniffer/issues/861
3413+
[#867]: https://github.com/squizlabs/PHP_CodeSniffer/issues/867
3414+
[#21005]: https://pear.php.net/bugs/bug.php?id=21005
3415+
[#21010]: https://pear.php.net/bugs/bug.php?id=21010
3416+
[#21011]: https://pear.php.net/bugs/bug.php?id=21011
33913417

33923418
## [2.5.0] - 2015-12-11
33933419
### Changed
3394-
- PHPCS will now look for a phpcs.xml file in parent directories as well as the current directory (request #626)
3420+
- PHPCS will now look for a phpcs.xml file in parent directories as well as the current directory (request [#626])
33953421
- PHPCS will now use a phpcs.xml file even if files are specified on the command line
33963422
- This file is still only used if no standard is specified on the command line
3397-
- Added support for a phpcs.xml.dist file (request #583)
3423+
- Added support for a phpcs.xml.dist file (request [#583])
33983424
- If both a phpcs.xml and phpcs.xml.dist file are present, the phpcs.xml file will be used
3399-
- Added support for setting PHP ini values in ruleset.xml files (request #560)
3425+
- Added support for setting PHP ini values in ruleset.xml files (request [#560])
34003426
- Setting the value of the new ini tags to name="memory_limit" value="32M" is the same as -d memory_limit=32M
34013427
- Added support for one or more bootstrap files to be run before processing begins
34023428
- Use the --bootstrap=file,file,file command line argument to include bootstrap files
@@ -3407,16 +3433,30 @@ Additionally, thanks to [Alexander Turek][@derrabus] for consulting on the repo
34073433
- Thanks to [Arnout Boks][@aboks] for the patch
34083434

34093435
### Fixed
3410-
- Fixed bug #660 : Syntax checks can fail on Windows with PHP5.6
3411-
- Fixed bug #784 : $this->trait is seen as a T_TRAIT token
3412-
- Fixed bug #786 : Switch indent issue with short array notation
3413-
- Fixed bug #787 : SpacingAfterDefaultBreak confused by multi-line statements
3414-
- Fixed bug #797 : Parsing CSS url() value breaks further parsing
3415-
- Fixed bug #805 : Squiz.Commenting.FunctionComment.InvalidTypeHint on Scalar types on PHP7
3416-
- Fixed bug #807 : Cannot fix line endings when open PHP tag is not on the first line
3417-
- Fixed bug #808 : JS tokeniser incorrectly setting some function and class names to control structure tokens
3418-
- Fixed bug #809 : PHPCBF can break a require_once statement with a space before the open parenthesis
3419-
- Fixed bug #813 : PEAR FunctionCallSignature checks wrong indent when first token on line is part of a multi-line string
3436+
- Fixed bug [#660] : Syntax checks can fail on Windows with PHP5.6
3437+
- Fixed bug [#784] : $this->trait is seen as a T_TRAIT token
3438+
- Fixed bug [#786] : Switch indent issue with short array notation
3439+
- Fixed bug [#787] : SpacingAfterDefaultBreak confused by multi-line statements
3440+
- Fixed bug [#797] : Parsing CSS url() value breaks further parsing
3441+
- Fixed bug [#805] : Squiz.Commenting.FunctionComment.InvalidTypeHint on Scalar types on PHP7
3442+
- Fixed bug [#807] : Cannot fix line endings when open PHP tag is not on the first line
3443+
- Fixed bug [#808] : JS tokenizer incorrectly setting some function and class names to control structure tokens
3444+
- Fixed bug [#809] : PHPCBF can break a require_once statement with a space before the open parenthesis
3445+
- Fixed bug [#813] : PEAR FunctionCallSignature checks wrong indent when first token on line is part of a multi-line string
3446+
3447+
[#560]: https://github.com/squizlabs/PHP_CodeSniffer/issues/560
3448+
[#583]: https://github.com/squizlabs/PHP_CodeSniffer/issues/583
3449+
[#626]: https://github.com/squizlabs/PHP_CodeSniffer/issues/626
3450+
[#660]: https://github.com/squizlabs/PHP_CodeSniffer/pull/660
3451+
[#784]: https://github.com/squizlabs/PHP_CodeSniffer/issues/784
3452+
[#786]: https://github.com/squizlabs/PHP_CodeSniffer/issues/786
3453+
[#787]: https://github.com/squizlabs/PHP_CodeSniffer/issues/787
3454+
[#797]: https://github.com/squizlabs/PHP_CodeSniffer/issues/797
3455+
[#805]: https://github.com/squizlabs/PHP_CodeSniffer/issues/805
3456+
[#807]: https://github.com/squizlabs/PHP_CodeSniffer/issues/807
3457+
[#808]: https://github.com/squizlabs/PHP_CodeSniffer/issues/808
3458+
[#809]: https://github.com/squizlabs/PHP_CodeSniffer/issues/809
3459+
[#813]: https://github.com/squizlabs/PHP_CodeSniffer/issues/813
34203460

34213461
## [2.4.0] - 2015-11-24
34223462
### Changed

0 commit comments

Comments
 (0)