Skip to content
New issue

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

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

Already on GitHub? # to your account

Check X-Error when WS connection fails #2568

Merged
merged 3 commits into from
Feb 8, 2022
Merged

Check X-Error when WS connection fails #2568

merged 3 commits into from
Feb 8, 2022

Conversation

romasku
Copy link
Contributor

@romasku romasku commented Jan 27, 2022

No description provided.

@codecov
Copy link

codecov bot commented Jan 27, 2022

Codecov Report

Merging #2568 (1ff1759) into master (fe85b4d) will decrease coverage by 0.00%.
The diff coverage is 95.83%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2568      +/-   ##
==========================================
- Coverage   85.11%   85.10%   -0.01%     
==========================================
  Files          70       70              
  Lines       11942    11927      -15     
  Branches     2022     2017       -5     
==========================================
- Hits        10164    10151      -13     
- Misses       1402     1405       +3     
+ Partials      376      371       -5     
Impacted Files Coverage Δ
neuro-sdk/src/neuro_sdk/_jobs.py 92.21% <95.65%> (+0.09%) ⬆️
neuro-sdk/src/neuro_sdk/_core.py 95.94% <96.00%> (+3.53%) ⬆️
neuro-cli/src/neuro_cli/file_logging.py 78.94% <0.00%> (-5.27%) ⬇️
neuro-cli/src/neuro_cli/ael.py 52.42% <0.00%> (-0.73%) ⬇️

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 fe85b4d...1ff1759. Read the comment docs.

@romasku romasku merged commit 6cda836 into master Feb 8, 2022
@romasku romasku deleted the rs/ws-errors branch February 8, 2022 12:18
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants