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

Fix compare original and formatted #1915

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

0daryo
Copy link
Contributor

@0daryo 0daryo commented Oct 24, 2024

Describe the PR
Compare original and formatted.
FIx #1914

Relation issue
#1914

Additional context
Add any other context about the problem here.

// @Summary Add a new pet to the store
// @Description get string by ID
// @Summary Add a new pet to the store
// @Description get string by ID
Copy link
Contributor Author

@0daryo 0daryo Oct 24, 2024

Choose a reason for hiding this comment

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

Only space between // and @ should be formatted.
Before my fix. Test fails.

% go test -run TestFormat_Format -v                                                                                                                                     
=== RUN   TestFormat_Format
    format_test.go:16: 
                Error Trace:    format_test.go:16
                Error:          Should be true
                Test:           TestFormat_Format
--- FAIL: TestFormat_Format (0.00s)
FAIL
exit status 1

@sdghchj sdghchj merged commit 9969858 into swaggo:master Oct 25, 2024
10 checks passed
@0daryo
Copy link
Contributor Author

0daryo commented Oct 29, 2024

@sdghchj
Thank you for merging!
Is it possible to use this fix in v2 release?

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.

Skipping overwrite in some case but should do.
2 participants