Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add CLI #42

Closed
1 task done
Snugug opened this issue Aug 28, 2015 · 8 comments
Closed
1 task done

Add CLI #42

Snugug opened this issue Aug 28, 2015 · 8 comments

Comments

@Snugug
Copy link
Member

Snugug commented Aug 28, 2015

Make Sass Lint run-able via a command line command. The CLI Branch is where work is being done on this feature.

  • Do not fail on error

Feedback Requested

What options should be available from the command line?

@davidgilbertson
Copy link

+1

Could it maybe be added to the readme that there isn't a CLI yet? I'm guessing you're about to have thousands of eager devs trying to get it working before realising that there is no CLI.

Also, this whole sass-lint thing is awesome, congrats on the 1.0 release!

@Snugug Snugug added this to the 1.1.0 milestone Aug 30, 2015
@Snugug
Copy link
Member Author

Snugug commented Aug 30, 2015

@Snugug
Copy link
Member Author

Snugug commented Aug 30, 2015

Track progress on the CLI Branch
https://github.com/sasstools/sass-lint/tree/cli

@davidgilbertson
Copy link

Nice.

My thoughts on command line options would be to more-or-less mirror scss-lint. Of those the two I can imagine using would be:

  • --config so that I can define a config file other than the default.
  • --exclude
  • I'm tempted to say a --files option to override the glob in the config file, but can't think of a use case.

@Snugug
Copy link
Member Author

Snugug commented Aug 30, 2015

So currently, this is how the CLI is shaping up:

  • sass-lint will use whatever is in the config file
  • sass-lint '<file or glob'> will override the file or glob from the config file
  • --no-exit will prevent the CLI from failing if there's an error

The config currently calls exclude ignore; I can swap them if need be and add that as an option.

I'm going to wait on #47 to land before providing a CLI for custom config.

@Snugug
Copy link
Member Author

Snugug commented Aug 30, 2015

Custom Config file landed in the CLI branch, so this'll be part of the CLI when released

@davidgilbertson
Copy link

Sounds perfect. Do you have a ballpark ETA on this? Weeks, months?

@Snugug
Copy link
Member Author

Snugug commented Aug 30, 2015

Probably days. It's working, just needs final tweaks.

On Aug 30, 2015, at 6:49 PM, David Gilbertson notifications@github.com wrote:

Sounds perfect. Do you have a ballpark ETA on this? Weeks, months?


Reply to this email directly or view it on GitHub.

Snugug added a commit that referenced this issue Aug 31, 2015
Verbose, Config, and Ignore

Relates to #42
donabrams pushed a commit to donabrams/sass-lint that referenced this issue Nov 19, 2015
Verbose, Config, and Ignore

Relates to sasstools#42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants