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

Updated to handle cases where 'body' is not present in the YAML file … #2973

Merged

Conversation

Shimizu1111
Copy link
Contributor

I have addressed GitHub Issue #2952
Updated to handle cases where 'body' is not present in the YAML file without causing an error.

@Franogales
Copy link

how can I replicate the error? I usualy validate if the body != nil. why is this needed? what would be the output?

@Shimizu1111 Shimizu1111 force-pushed the issue-2952-swagger-diff-runtime-error branch from ca9e910 to e2f4ef9 Compare October 4, 2023 08:18
@Shimizu1111
Copy link
Contributor Author

See #2952 for contents and how to reproduce the error.

…without causing an error.

Signed-off-by: Shimizu1111 <foritemaqua@gmail.com>
@casualjim casualjim force-pushed the issue-2952-swagger-diff-runtime-error branch from e2f4ef9 to 2296149 Compare November 26, 2023 08:19
@casualjim casualjim merged commit 080a34d into go-swagger:master Nov 27, 2023
15 checks passed
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.

None yet

3 participants