yujinakayama / atom-lint

Obsolete: Generic code linting support for Atom
https://atom.io/packages/atom-lint
MIT License
111 stars 33 forks source link

Feature/csslint options #55

Closed elrolito closed 10 years ago

elrolito commented 10 years ago

Can now add an array of CSSLint options to either ignore, errors, or warnings command line flags in config.json, which should resolve issue #54.

'atom-lint':
  'csslint':
    'path': '/usr/local/bin/csslint'
    'rules':
      'ignore': ['adjoining-classes']
elrolito commented 10 years ago

Going to tweak some more actually...

yujinakayama commented 10 years ago

This needs to be mentioned in the README.

yujinakayama commented 10 years ago

Also, please squach all the commits into one.

elrolito commented 10 years ago

updated.

yujinakayama commented 10 years ago

Thanks!

By the way, more fundamental solution is discussed in CSSLint itself (https://github.com/CSSLint/csslint/issues/449#issuecomment-33072445) though it has not yet implemented. So I consider this is a temporary solution and I'll remove this from atom-lint once the feature is introduced to CSSLint.

elrolito commented 10 years ago

No problem, glad I could help. And yes, proper .csslintrc (global and local) file support would be ideal, and I agree, this is just a temporary solution. Thanks.