? Pending

User tests: Successful: Unsuccessful:

avatar photodude
photodude
24 Nov 2017

Pull Request for Issue code style fixes

Summary of Changes

  • Tabs must be used to indent lines; spaces are not allowed
  • Whitespace found at end of line

Automatically fixed with Joomla code standards 2.0.0 PHPCS2-alpha2 fixers

None of the manual only fixes have been applied

Testing Instructions

Merge by code review

Expected result

code style has been applied as listed above, old code style on drone does not error.

Actual result

code style had not been applied. Autofixers from the Joomla code standards 2.0.0 PHPCS2 alpha2 were used to implement fixable code style

Documentation Changes Required

none

avatar photodude photodude - open - 24 Nov 2017
avatar photodude photodude - change - 24 Nov 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 Nov 2017
Category Administration com_login
avatar photodude photodude - change - 24 Nov 2017
Title
PHPCS2 -autofixers
[CS] Code Style fixes for administrator/components/com_login/
avatar photodude photodude - edited - 24 Nov 2017
avatar photodude photodude - change - 24 Nov 2017
Labels Added: ?
avatar Quy Quy - test_item - 24 Nov 2017 - Tested successfully
avatar Quy
Quy - comment - 24 Nov 2017

I have tested this item successfully on 1e0dbb5


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/18842.

avatar wilsonge wilsonge - change - 26 Nov 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-11-26 23:03:24
Closed_By wilsonge
avatar wilsonge wilsonge - close - 26 Nov 2017
avatar wilsonge wilsonge - merge - 26 Nov 2017
avatar wilsonge
wilsonge - comment - 26 Nov 2017

@photodude might be worth updating your link to this file in your squizlabs/PHP_CodeSniffer#1760 to point to the 3.8.2 tag :)

avatar photodude
photodude - comment - 27 Nov 2017

Thanks merges @wilsonge, I have updated the link in the PHPCS PR as suggested.

Add a Comment

Login with GitHub to post a comment