Skip to content

Documentation: include . in the example commands - #88

Closed
olafurpg wants to merge 1 commit into
scip-ruby/masterfrom
olafurpg/readme2
Closed

olafurpg wants to merge 1 commit into
scip-ruby/masterfrom
olafurpg/readme2

Conversation

@olafurpg

Copy link
Copy Markdown
Contributor

I wasn't able to get the typechecking to run without the .

❯ scip-ruby --index-file index.scip
You must pass either `-e` or at least one folder or ruby file.

Typechecker for Ruby
Usage:
  sorbet [OPTION...] <path 1> <path 2> ...

  -e string      Parse an inline ruby string (default: "")
  -q, --quiet    Silence all non-critical errors
  -v, --verbose  Verbosity level [0-3]
  -h             Show short help
      --help     Show long help
      --version  Show version

❯ echo $?
1

I wasn't able to get the typechecking to run without the `.`

```
❯ scip-ruby --index-file index.scip
You must pass either `-e` or at least one folder or ruby file.

Typechecker for Ruby
Usage:
  sorbet [OPTION...] <path 1> <path 2> ...

  -e string      Parse an inline ruby string (default: "")
  -q, --quiet    Silence all non-critical errors
  -v, --verbose  Verbosity level [0-3]
  -h             Show short help
      --help     Show long help
      --version  Show version

❯ echo $?
1
```
@varungandhi-src

varungandhi-src commented Aug 25, 2022

Copy link
Copy Markdown
Contributor

Do you run into this issue with bundle exec too or just direct invocation? For bundle exec, it does seem to work fine for me. 🤔

bash-5.1$ bundle exec scip-ruby --index-file index.scip --gem-metadata "brew@3.5.9"
No errors! Great job.

Same for direct invocation of scip-ruby:

❯ ~/Code/scip-ruby/bazel-bin/main/scip-ruby --index-file index.scip --gem-metadata "brew@3.5.9"
No errors! Great job.

What scip-ruby version are you using?

@varungandhi-src

Copy link
Copy Markdown
Contributor

Can you share the output of env too?

@varungandhi-src
varungandhi-src force-pushed the scip-ruby/master branch 4 times, most recently from 7f3acac to da9285d Compare August 31, 2022 04:36
@varungandhi-src

Copy link
Copy Markdown
Contributor

Ah, OK. Narrowed this down. This seems to be needed when the sorbet/ folder is not present. I will create an updated PR to document this, closing this one.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants