digitpaint/roger_scsslint

Lint SCSS files within Roger

roger_scsslint's Issues

Give feedback on what's being linted

Currently you get one big blob of feedback at the end instead of continous feedback.

Deal with scss-lint exit codes

scsslint returns 1 for warnings, and 2 for errors. Warnings should not trigger a CI fail, but CI systems obviously treat 1 as a failure.

Add option to supply :match and :skip paramaters

So we can specify which files we want to be tested; the behaviour should be the same as with release processors (they use this function: https://github.com/DigitPaint/roger/blob/master/lib/roger/release.rb#L195)

Rename to roger_scsslint

Also register it as :scsslint

Make filenames match with module structure

Rules:

• Only CamelCase underscores
• Only make modules from directory structures

Fixes:

• Rename base module to RogerScsslint (Also see #1)
• Fix roger issue DigitPaint/roger#6
• Rename RogerScssLinter::Generators:: ScssLinterGenerator to RogerScsslint::Generator

Add digitpaint to gem ownership

Change default config url

Use the default config from a DP repository. Or even better, include a default config in the gem.

Discuss the options.

Add tests

The linter and the generator do not have tests

Add tests for all configuration options

Prevent the linter from using the library directly

SCSS-lint overwrites certain SASS functions. This will BREAK your implementation. WE need to invoke the linter through the commandline instead.

The test directory is messy

The test directory contains data as well as the tests; please put the test data in a separate folder

Add travis config

Also add travis badges t readme