?
avatar zero-24
zero-24
23 Jan 2017

Steps to reproduce the issue

https://travis-ci.org/joomla/joomla-cms/jobs/194531446

Expected result

Cs test passes

Actual result

CS tests fails ...

avatar zero-24 zero-24 - open - 23 Jan 2017
avatar joomla-cms-bot joomla-cms-bot - change - 23 Jan 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 23 Jan 2017
avatar C-Lodder
C-Lodder - comment - 23 Jan 2017

tests need to be re-written to cater for BS4

avatar zero-24
zero-24 - comment - 23 Jan 2017

tests need to be re-written to cater for BS4

? Should have been done before we merge it into the core?

First Part: #13723

avatar zero-24
zero-24 - comment - 23 Jan 2017

Seccond part: #13724

@wilsonge please review and merge so we can be sure we are ok.

avatar photodude
photodude - comment - 25 Jan 2017

@C-Lodder What in the Code standards need to be rewritten?

I'm not seeing anything other than files that are no longer excluded that have code style issues. (some of these can be fixed with the auto fixers in the PHPCS 2 version of the coding standards See joomla/coding-standards#143 for items relating to the upcoming Alpha release)

avatar C-Lodder
C-Lodder - comment - 25 Jan 2017

@photodude - Code Standards for Bootstrap?

Were there any for BS2? If so, could I have a link please to get a better idea

avatar photodude
photodude - comment - 25 Jan 2017

@C-Lodder our current (needing revision) code standards for JS are here in the manual (until the gh-pages find a new home) http://joomla.github.io/coding-standards/?coding-standards/chapters/javascript.md

but JS files and mixed php/html files are not CI checked for code standards. We currently only check pure PHP files (and not all of them).

avatar C-Lodder
C-Lodder - comment - 25 Jan 2017

Ah didn't know the CS page had requirements for CSS. I'll make the necessary updates for the SCSS tomorrow.

avatar photodude
photodude - comment - 25 Jan 2017

Yep we have CS items for
PHP Code
XML
and the Clientside Syntaxs
HTML
CSS
JavaScript

avatar C-Lodder
C-Lodder - comment - 26 Jan 2017
avatar zero-24 zero-24 - change - 5 Feb 2017
Status New Closed
Closed_Date 0000-00-00 00:00:00 2017-02-05 20:19:44
Closed_By zero-24
avatar zero-24 zero-24 - close - 5 Feb 2017

Add a Comment

Login with GitHub to post a comment