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.

Sign up for GitHub

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

Jump to bottom

CSpell does not check all files if one is missing with --file-list #2285

Closed
3 tasks done
Jason3S opened this issue Jan 20, 2022 · 1 comment · Fixed by #2286
Closed
3 tasks done

CSpell does not check all files if one is missing with --file-list #2285

Jason3S opened this issue Jan 20, 2022 · 1 comment · Fixed by #2286
Labels
bug Needs Investigating new issue

Comments

@Jason3S
Copy link
Collaborator

Jason3S commented Jan 20, 2022

Info

Kind of Issue

  • runtime - command-line tools
  • crash / error

Which Tool or library

  • cspell -- the command-line spelling tool

Which Version

Version: 5.15.1

Bug Description

Describe the bug

When using a command like:

git diff --name-only | cspell --no-must-find-files --file-list stdin

Like:

Linter Error: Error reading file: "/Users/jason/projects/cspell5/codecov.yaml"

and a file had been deleted, an error will be produced.

Expected Behavior

That is should not error when --no-must-find-files is specified.
Jason3S added a commit that referenced this issue Jan 20, 2022
@Jason3S
fix: #2285
66bd856 
- Continue processing the list of files even if some are missing.
- Report on any missing files.
- error code is based upon `--no-must-file-files` option.
@Jason3S Jason3S linked a pull request Jan 20, 2022 that will close this issue
fix: Handle missing files when spell checking from a file list. #2286
Merged
@Jason3S Jason3S mentioned this issue Jan 20, 2022
Merged
Jason3S added a commit that referenced this issue Jan 20, 2022
@Jason3S
fix: Handle missing files when spell checking from a file list. (#2286)
fd1e7e2 
fix: #2285 

- Continue processing the list of files even if some are missing.
- Report on any missing files.
- error code is based upon `--no-must-file-files` option.
@github-actions
Copy link
Contributor

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Needs Investigating new issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Handle missing files when spell checking from a file list. streetsidesoftware/cspell
1 participant
@Jason3S