Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Continuous Integration #2

Open
5 tasks done
ndarville opened this issue Apr 4, 2014 · 6 comments
Open
5 tasks done

Continuous Integration #2

ndarville opened this issue Apr 4, 2014 · 6 comments
Labels
enhancement

Comments

@ndarville
Copy link
Owner

  • SCSS linting
    • .scss-lint.yml is not respected, locally and on travis
  • CSS linting
  • HTML linting
  • JS linting
  • Return error code on critical failures
@ndarville
Copy link
Owner Author

htmlhint and jshint return an SSL-cert-validation error on installation, so I need to find an alternative.

@ndarville
Copy link
Owner Author

Fixed the node.js crap, so jshint is working now. htmlhint seems borked, though.

@ndarville
Copy link
Owner Author

htmlhint has now been fixed by the developer, so it’s been enabled and working.

@ndarville
Copy link
Owner Author

Maybe patching scss-lint will fix the problem.

@ndarville
Copy link
Owner Author

scss-lint now works after updating the versions and using the new linter names.

Travis doesn’t fail any builds with HTML/(S)CSS errors, however.

@ndarville
Copy link
Owner Author

This error on Travis seems broken:

Syntax error: Undefined variable: "$fontSize".
        on line 2 of static/css/_base.scss
  Use --trace for backtrace.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ndarville