? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
24 Feb 2020

The scc-lint is not running :(

This PR fixes as many issues as I can according to our codestyle rules found in scss-lint.yml

Code review and check scss compiles

Feel free to fix other files

avatar brianteeman brianteeman - open - 24 Feb 2020
avatar brianteeman brianteeman - change - 24 Feb 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 Feb 2020
Category Administration Templates (admin)
avatar C-Lodder
C-Lodder - comment - 24 Feb 2020

How are you trying to run the linter?

avatar brianteeman
brianteeman - comment - 24 Feb 2020

these changes were made by running the linter locally and then following the reports

avatar brianteeman
brianteeman - comment - 24 Feb 2020

The scc-lint is not running :(

I meant its not running on pull requests see #26800

avatar C-Lodder
C-Lodder - comment - 24 Feb 2020

Oh Hound. Hmm I'll try and look into this

avatar brianteeman
brianteeman - comment - 24 Feb 2020

Thanks - I would really appreciate it as my local lint report is humungous

avatar C-Lodder
C-Lodder - comment - 24 Feb 2020

@brianteeman ROOT/scss-lint.yml

Replace:

scss_files:
  "administrator/components/com_media/resources/**/*.scss"
  "administrator/templates/**/*.scss"
  "build/media_source/**/*.scss"
  "templates/**/*.scss"

with:

scss_files: [
  "administrator/components/com_media/resources/**/*.scss",
  "administrator/templates/**/*.scss",
  "build/media_source/**/*.scss",
  "templates/**/*.scss"
]

You'll probably want to undo all your changes in this PR though as they go against the linter rules :)

avatar C-Lodder
C-Lodder - comment - 24 Feb 2020

The linter wasn't actualy working at all. It threw an error due to an invalid YML file on my fork but doesn't appear to do so on here for some reason.

avatar brianteeman brianteeman - change - 24 Feb 2020
Labels Added: ?
avatar brianteeman brianteeman - change - 25 Feb 2020
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2020-02-25 12:10:00
Closed_By brianteeman
avatar brianteeman brianteeman - close - 25 Feb 2020

Add a Comment

Login with GitHub to post a comment