sds / scss-lint

96%

master: 96%

LAST BUILD ON BRANCH universal-newline

branch: universal-newline

CHANGE BRANCH

x

Reset

• universal-newline
• master
• v0.40.0
• v0.40.1
• v0.41.0
• v0.42.0
• trailing-semicolon-important
• bang-format
• v0.42.1
• v0.42.2
• v0.43.0
• pseudoselector
• v0.43.1
• v0.43.2
• v0.44.0
• single-line-per-property-nesting
• relax-rake-dependency-version
• v0.45.0
• PrivateNamingConvention
• linter-gem-spec
• tap-reporter
• private-naming-edge-cases
• v0.46.0
• private-naming-convention-bugs
• tap-reporter-format
• v0.47.0
• tap-report-path-fix
• v0.47.1
• v0.48.0
• v0.49.0
• v0.50.0
• v0.50.1
• v0.50.2
• v0.50.3
• v0.51.0
• v0.52.0
• v0.53.0
• v0.54.0
• v0.55.0
• v0.56.0
• v0.57.0
• v0.57.1
• v0.58.0
• v0.59.0

Build # 1113

Build Type

push

travis-ci

Committed by lencioni

Commit Message

Enable universal newline when reading scss files

SCSS-Lint seems to choke on a file that uses windows-style newlines with
carriage returns. (To type a windows-style newline with carriage return
in Vim, press <Ctrl>+v<Ctrl>+m in insert mode. This will show up as ^M.)

> SCSSLint::Linter::Indentation raised unexpected error linting file
> ./path/to/file.scss: 'undefined method `[]' for nil:NilClass'

To avoid this, we can encode the file contents using String#encode's
:universal_newline option, which

> Replaces CRLF (“rn”) and CR (“r”) with LF (“n”) if value is true.

http://ruby-doc.org/core-2.2.0/String.html#method-i-encode

I thought that maybe we wouldn't need the force_encoding anymore after
this change, but that was not the case since the test that was added
with that change started failing once I removed the force_encoding.

Fixes #700.

Run Details

2646 of 2751 relevant lines covered (96.18%)

1391.0 hits per line