Skip to content

Handle glob errors instead of crashing #224

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

Merged
merged 1 commit into from
Apr 17, 2020
Merged

Handle glob errors instead of crashing #224

merged 1 commit into from
Apr 17, 2020

Conversation

skovhus
Copy link
Collaborator

@skovhus skovhus commented Apr 17, 2020

Previously, when we failed to resolve the glob for shell files we crashed. Instead, we should just show a warning – the server can still be useful.

#223

@skovhus skovhus force-pushed the handle-glob-error branch from 8533f4d to 1d8a7df Compare April 17, 2020 18:47
@codecov
Copy link

codecov bot commented Apr 17, 2020

Codecov Report

Merging #224 into master will increase coverage by 0.17%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #224      +/-   ##
==========================================
+ Coverage   70.56%   70.73%   +0.17%     
==========================================
  Files          18       18              
  Lines         513      516       +3     
  Branches       81       81              
==========================================
+ Hits          362      365       +3     
  Misses        130      130              
  Partials       21       21              
Impacted Files Coverage Δ
server/src/analyser.ts 80.82% <100.00%> (+0.40%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cca9a26...1d8a7df. Read the comment docs.

@skovhus skovhus merged commit f07d4c0 into master Apr 17, 2020
@skovhus skovhus deleted the handle-glob-error branch April 17, 2020 19:48
@skovhus skovhus mentioned this pull request Apr 18, 2020
akurtakov added a commit to akurtakov/shellwax that referenced this pull request Apr 21, 2020
Changelog of the server:
1.13.1

    Gracefully handle glob failures
(bash-lsp/bash-language-server#224,
bash-lsp/bash-language-server#226)
    Maintenance
(bash-lsp/bash-language-server#222,
bash-lsp/bash-language-server#225)

1.13.0

    Upgrade vscode-languageserver from 5 to 6
(bash-lsp/bash-language-server#217)

1.12.0

    Completion handler improvements: remove duplicates, include symbols
from other files, ensure that programs found on the paths are actually
executable (bash-lsp/bash-language-server#215)

1.11.3

    Recover from file reading errors
(bash-lsp/bash-language-server#211)


Signed-off-by: Alexander Kurtakov <[email protected]>
akurtakov added a commit to akurtakov/shellwax that referenced this pull request Apr 21, 2020
Changelog of the server:
1.13.1

    Gracefully handle glob failures
(bash-lsp/bash-language-server#224,
bash-lsp/bash-language-server#226)
    Maintenance
(bash-lsp/bash-language-server#222,
bash-lsp/bash-language-server#225)

1.13.0

    Upgrade vscode-languageserver from 5 to 6
(bash-lsp/bash-language-server#217)

1.12.0

    Completion handler improvements: remove duplicates, include symbols
from other files, ensure that programs found on the paths are actually
executable (bash-lsp/bash-language-server#215)

1.11.3

    Recover from file reading errors
(bash-lsp/bash-language-server#211)


Signed-off-by: Alexander Kurtakov <[email protected]>
akurtakov added a commit to eclipse-shellwax/shellwax that referenced this pull request Apr 21, 2020
Changelog of the server:
1.13.1

    Gracefully handle glob failures
(bash-lsp/bash-language-server#224,
bash-lsp/bash-language-server#226)
    Maintenance
(bash-lsp/bash-language-server#222,
bash-lsp/bash-language-server#225)

1.13.0

    Upgrade vscode-languageserver from 5 to 6
(bash-lsp/bash-language-server#217)

1.12.0

    Completion handler improvements: remove duplicates, include symbols
from other files, ensure that programs found on the paths are actually
executable (bash-lsp/bash-language-server#215)

1.11.3

    Recover from file reading errors
(bash-lsp/bash-language-server#211)


Signed-off-by: Alexander Kurtakov <[email protected]>
akurtakov added a commit to akurtakov/shellwax that referenced this pull request May 27, 2020
Language server changelog:
1.16.1

    Fix brace expansion bug
(bash-lsp/bash-language-server#240)
    Do not crash if bash is not installed
(bash-lsp/bash-language-server#242)

1.16.0

    Improved completion handler for parameter expansions
(bash-lsp/bash-language-server#237)

1.15.0

    Use comments above symbols for documentation
(bash-lsp/bash-language-server#234,
bash-lsp/bash-language-server#235)

1.14.0

    onHover and onCompletion documentation improvements
(bash-lsp/bash-language-server#230)
    support 0/1 as values for HIGHLIGHT_PARSING_ERRORS
(bash-lsp/bash-language-server#231)

1.13.1

    Gracefully handle glob failures
(bash-lsp/bash-language-server#224,
bash-lsp/bash-language-server#226)
    Maintenance
(bash-lsp/bash-language-server#222,
bash-lsp/bash-language-server#225)


Signed-off-by: Alexander Kurtakov <[email protected]>
akurtakov added a commit to eclipse-shellwax/shellwax that referenced this pull request May 27, 2020
Language server changelog:
1.16.1

    Fix brace expansion bug
(bash-lsp/bash-language-server#240)
    Do not crash if bash is not installed
(bash-lsp/bash-language-server#242)

1.16.0

    Improved completion handler for parameter expansions
(bash-lsp/bash-language-server#237)

1.15.0

    Use comments above symbols for documentation
(bash-lsp/bash-language-server#234,
bash-lsp/bash-language-server#235)

1.14.0

    onHover and onCompletion documentation improvements
(bash-lsp/bash-language-server#230)
    support 0/1 as values for HIGHLIGHT_PARSING_ERRORS
(bash-lsp/bash-language-server#231)

1.13.1

    Gracefully handle glob failures
(bash-lsp/bash-language-server#224,
bash-lsp/bash-language-server#226)
    Maintenance
(bash-lsp/bash-language-server#222,
bash-lsp/bash-language-server#225)


Signed-off-by: Alexander Kurtakov <[email protected]>
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.

1 participant