? Success

User tests: Successful: Unsuccessful:

avatar rdeutz
rdeutz
14 Jan 2015

Added the missing a close tag

avatar rdeutz rdeutz - open - 14 Jan 2015
avatar jissues-bot jissues-bot - change - 14 Jan 2015
Labels Added: ?
avatar Hackwar
Hackwar - comment - 14 Jan 2015

@test success via codereview. Could you please add one empty line at the end of the file?

avatar zero-24 zero-24 - change - 14 Jan 2015
Category Code style
avatar zero-24 zero-24 - change - 14 Jan 2015
The description was changed
Status Pending Ready to Commit
Easy No Yes
avatar zero-24
zero-24 - comment - 14 Jan 2015

RTC


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

avatar zero-24 zero-24 - change - 14 Jan 2015
Status Ready to Commit Pending
avatar zero-24
zero-24 - comment - 14 Jan 2015

moving back to pending as the missing line that was find by @Hackwar


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

avatar brianteeman brianteeman - change - 14 Jan 2015
Labels Added: ?
avatar rdeutz
rdeutz - comment - 14 Jan 2015

How I get this fixed? I did a commit with an empty line (amend) it with the one I already did. In my branch the file has 193 lines so there is a new line at the end of the file. If I delete the local file and pull from my branch I get a file with an empty line at the end. So anything looks fine. Does github not showing the empty line at the end of the file?


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

avatar Hackwar
Hackwar - comment - 14 Jan 2015

It's okay the way it is. If there is no linebreak after the last command, github shows a red error behind the last command. So this is fixed and ready to be merged.

avatar zero-24 zero-24 - change - 14 Jan 2015
Status Pending Ready to Commit
avatar zero-24
zero-24 - comment - 14 Jan 2015

Thanks @Hackwar moving back to RTC


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

avatar zero-24 zero-24 - close - 15 Jan 2015
avatar roland-d roland-d - close - 15 Jan 2015
avatar roland-d roland-d - change - 15 Jan 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-01-15 20:07:57
avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment