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.

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

ResponseStatusException added #82

Merged
merged 3 commits into from
Mar 11, 2024
Merged

ResponseStatusException added #82

merged 3 commits into from
Mar 11, 2024

Conversation

dukris
Copy link
Collaborator

@dukris dukris commented Mar 11, 2024

@h1alexbel take a look, please

Closes #80


PR-Codex overview

This PR updates error messages in test cases and exception handling to include HTTP status codes for better clarity and consistency.

Detailed summary

  • Updated error messages in test cases to include HTTP status codes for better clarity.
  • Changed exception handling to include HTTP status codes for consistency.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@dukris dukris requested a review from h1alexbel March 11, 2024 14:45
Copy link
Collaborator

@h1alexbel h1alexbel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@h1alexbel
Copy link
Collaborator

@rultor merge

@rultor
Copy link
Collaborator

rultor commented Mar 11, 2024

@rultor merge

@h1alexbel Thanks for your request; @hizmailovich please confirm this.

@dukris
Copy link
Collaborator Author

dukris commented Mar 11, 2024

@rultor merge

@rultor
Copy link
Collaborator

rultor commented Mar 11, 2024

@rultor merge

@hizmailovich OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit 385414f into master Mar 11, 2024
6 checks passed
@rultor
Copy link
Collaborator

rultor commented Mar 11, 2024

@rultor merge

@hizmailovich Done! FYI, the full log is here (took me 4min)

@dukris dukris deleted the 80 branch March 12, 2024 07:21
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.

Can be replaced with ResponseStatusException
3 participants