Yaml parser crashes on missing indentation

vadimdemedes / trevor

🚦 Your own mini Travis CI to run tests locally

MIT License

2.11k stars 51 forks source link

Yaml parser crashes on missing indentation #40

Open DirtyF opened 7 years ago

DirtyF commented 7 years ago

❯ node -v
v7.5.0

❯ more .travis.yml
language: node_js
node_js:
- 7
- 6
cache:
  directories:
  - node_modules
install:
- npm install
script:
- npm test

❯ travis lint
Hooray, .travis.yml looks valid :)

❯ trevor

/usr/local/lib/node_modules/trevor/node_modules/yamljs/lib/Parser.js:268
        throw new ParseException('Unable to parse.', this.getRealCurrentLineNb() + 1, this.currentLine);
        ^
Error: Unable to parse.

vadimdemedes commented 7 years ago

Looks like the yaml parser Trevor uses (yamljs) doesn't like the lack of indentation for - items. Thanks for reporting this!

vadimdemedes commented 7 years ago

Hm, I was unable to reproduce the error with your .travis.yml. Could you upload your exact .travis.yml?

DirtyF commented 7 years ago

@vadimdemedes It' here: https://github.com/sudweb/talks/blob/master/.travis.yml