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

Running eslint in STDIN mode on an empty file prints weirdly formated help text #9515

Closed
cpitclaudel opened this Issue Oct 25, 2017 · 2 comments

Comments

Projects
None yet
2 participants
@cpitclaudel

cpitclaudel commented Oct 25, 2017

  • ESLint Version: 3.7.1
  • Node Version: 8.7.0
  • npm Version: 5.4.2

What parser (default, Babel-ESLint, etc.) are you using? default

Please show your full configuration: None

Running eslint in STDIN mode on an empty file prints weirdly formated help text:

$ touch a.js
$ eslint --format=checkstyle --stdin --stdin-filename a.js < a.js
eslint [options] file.js [file.js] [dir]

Basic configuration:
  -c, --config path::String

      Use
      configuration
      from
      this
      file
      or
      shareable
      config
  --no-eslintrc

      Disable
      use
      of
      configuration
      from
      .eslintrc
  --env [String]

      Specify
      environments
  --ext [String]

      Specify
      JavaScript
      file
      extensions
      -
      default:
      .js
[… lots more output …]
$ echo $?
0

Is this expected?

@not-an-aardvark

This comment has been minimized.

Show comment
Hide comment
@not-an-aardvark

not-an-aardvark Oct 25, 2017

Member

Nope. Thanks for the report.

Member

not-an-aardvark commented Oct 25, 2017

Nope. Thanks for the report.

not-an-aardvark added a commit that referenced this issue Oct 25, 2017

Fix: allow linting the empty string from stdin (fixes #9515)
This fixes an issue where `cli` would treat an empty-string argument as
missing.

@Aladdin-ADD Aladdin-ADD closed this in #9517 Oct 26, 2017

Aladdin-ADD added a commit that referenced this issue Oct 26, 2017

Fix: allow linting the empty string from stdin (fixes #9515) (#9517)
This fixes an issue where `cli` would treat an empty-string argument as
missing.
@cpitclaudel

This comment has been minimized.

Show comment
Hide comment
@cpitclaudel

cpitclaudel commented Oct 26, 2017

Thanks!

@eslint eslint bot locked and limited conversation to collaborators Apr 25, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.