Skip to content

Show human-readable errors in CLI client #55

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
Merged
forman merged 12 commits into from
Oct 20, 2025
Merged

Show human-readable errors in CLI client #55

forman merged 12 commits into from
Oct 20, 2025

Conversation

@forman
Copy link
Collaborator

@forman forman commented Oct 16, 2025

[Description of PR]

Checklist (strike out non-applicable):

  • Changes documented in CHANGES.md
  • Related issue exists and is referred to in the PR description and CHANGES.md
  • Added docstrings and API docs for any new/modified user-facing classes and functions
  • Changes/features documented in docs/*
  • Unit-tests adapted/added for changes/features
  • Test coverage remains or increases (target 100%)

forman added 11 commits October 16, 2025 13:20
@forman
raising TransportException
119d013
@forman
Merge branch 'main' into forman-54-handle_errors_in_client_cli
d657d1c
@forman
cont on generating exec requests
4783bfa
@forman
Handling TransportException now
9ab6938
@forman
fix tests
29b8a60
@forman
fix tests
289ffb6
@forman
Merge branch 'forman-x-fix-exports' into forman-54-handle_errors_in_c…
9e5edfb 
…lient_cli
@forman
fix tests
5456d16
@forman
- In s2gos_client:
8a9ef4a 
  - Improved error handling (#54)
  - Renamed `ClientException` into `ClientError`.
  - Renamed `TransportException` into `TransportError`.
@forman
- In s2gos_client:
f0a0b8a 
  - Improved error handling (#54)
  - Renamed `ClientException` into `ClientError`.
  - Renamed `TransportException` into `TransportError`.
@forman
Updated docs and foxed another test for CI
7c1d1e8
@codecov
Copy link

codecov bot commented Oct 20, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@forman forman marked this pull request as ready for review October 20, 2025 15:18
@forman forman merged commit c9e0c53 into main Oct 20, 2025
2 checks passed
@forman forman deleted the forman-54-handle_errors_in_client_cli branch October 20, 2025 15:18
@forman forman mentioned this pull request Oct 20, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@forman