-
Notifications
You must be signed in to change notification settings - Fork 0
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
Initial #2
Merged
Merged
Initial #2
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Added NewContinue for HTTP 100 Continue - Added NewSwitchingProtocols for HTTP 101 Switching Protocols - Added NewOK for HTTP 200 OK - Added NewCreated for HTTP 201 Created - Added NewAccepted for HTTP 202 Accepted - Added NewNoContent for HTTP 204 No Content - Added NewMovedPermanently for HTTP 301 Moved Permanently - Added NewFound for HTTP 302 Found - Added NewNotModified for HTTP 304 Not Modified - Added NewUnauthorized for HTTP 401 Unauthorized - Added NewForbidden for HTTP 403 Forbidden - Added NewMethodNotAllowed for HTTP 405 Method Not Allowed - Added NewConflict for HTTP 409 Conflict - Added NewUnprocessableEntity for HTTP 422 Unprocessable Entity - Added NewNotImplemented for HTTP 501 Not Implemented - Added NewBadGateway for HTTP 502 Bad Gateway - Added NewServiceUnavailable for HTTP 503 Service Unavailable Add tests for all HTTP problem types - Added tests for HTTP 100 Continue - Added tests for HTTP 101 Switching Protocols - Added tests for HTTP 200 OK - Added tests for HTTP 201 Created - Added tests for HTTP 202 Accepted - Added tests for HTTP 204 No Content - Added tests for HTTP 301 Moved Permanently - Added tests for HTTP 302 Found - Added tests for HTTP 304 Not Modified - Added tests for HTTP 400 Bad Request - Added tests for HTTP 401 Unauthorized - Added tests for HTTP 403 Forbidden - Added tests for HTTP 404 Not Found - Added tests for HTTP 405 Method Not Allowed - Added tests for HTTP 409 Conflict - Added tests for HTTP 422 Unprocessable Entity - Added tests for HTTP 500 Internal Server Error - Added tests for HTTP 501 Not Implemented - Added tests for HTTP 502 Bad Gateway - Added tests for HTTP 503 Service Unavailable Add ProblemType definitions for various HTTP status codes - Added ProblemType for HTTP 100 Continue - Added ProblemType for HTTP 101 Switching Protocols - Added ProblemType for HTTP 200 OK - Added ProblemType for HTTP 201 Created - Added ProblemType for HTTP 202 Accepted - Added ProblemType for HTTP 203 Non-Authoritative Information - Added ProblemType for HTTP 204 No Content - Added ProblemType for HTTP 205 Reset Content - Added ProblemType for HTTP 206 Partial Content - Added ProblemType for HTTP 300 Multiple Choices - Added ProblemType for HTTP 301 Moved Permanently - Added ProblemType for HTTP 302 Found - Added ProblemType for HTTP 303 See Other - Added ProblemType for HTTP 304 Not Modified - Added ProblemType for HTTP 305 Use Proxy - Added ProblemType for HTTP 307 Temporary Redirect - Added ProblemType for HTTP 308 Permanent Redirect - Added ProblemType for HTTP 400 Bad Request - Added ProblemType for HTTP 401 Unauthorized - Added ProblemType for HTTP 403 Forbidden - Added ProblemType for HTTP 404 Not Found - Added ProblemType for HTTP 405 Method Not Allowed - Added ProblemType for HTTP 406 Not Acceptable - Added ProblemType for HTTP 409 Conflict - Added ProblemType for HTTP 410 Gone - Added ProblemType for HTTP 422 Unprocessable Entity - Added ProblemType for HTTP 500 Internal Server Error - Added ProblemType for HTTP 501 Not Implemented - Added ProblemType for HTTP 502 Bad Gateway - Added ProblemType for HTTP 503 Service Unavailable - Added ProblemType for HTTP 504 Gateway Timeout - Added ProblemType for HTTP 505 HTTP Version Not Supported
…blemtype.go`. - Updated `ProblemDetails` methods for creating instances in `problemdetails.go`. - Added unit tests for `ProblemDetails` creation functions in `problemdetails_test.go`.
- Defined `ProblemType` struct and various HTTP problem types in `problemtype.go`. - Updated `ProblemDetails` methods for creating instances in `problemdetails.go`. - Added unit tests for `ProblemDetails` creation functions in `problemdetails_test.go`. - Added examples to `internal/examples` - Updated README.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Welcome to the project! Verify your commit history (git log) to ensure it does NOT contain public email addresses!
stevensefton
requested changes
Oct 14, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See inline comment
Added test functions for methods with additional details and extensions: - TestNewWithDetail - TestNewWithExtension
- Added example of output in README.md
stevensefton
previously approved these changes
Oct 15, 2024
- Updated date on CHANGELOG.md
- Updated README.md to remove 'JSONJSON'
kbiedinger
approved these changes
Oct 15, 2024
- Updated README.md to remove 'JSONJSON'
ImranAliMohammed
approved these changes
Oct 15, 2024
neiljpowell
previously approved these changes
Oct 15, 2024
neiljpowell
approved these changes
Oct 15, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added badges