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 x tag groups #1

Merged
merged 119 commits into from
Jan 13, 2021
Merged

Fix x tag groups #1

merged 119 commits into from
Jan 13, 2021

Conversation

Nerzal
Copy link
Owner

@Nerzal Nerzal commented Jan 13, 2021

Describe the PR
e.g. add cool parser.

Relation issue
e.g. https://github.com/swaggo/swag/pull/118/files

Additional context
Add any other context about the problem here.

greenpart and others added 30 commits September 4, 2019 15:39
* add markdown description to operation
)

* add tag 'swaggerignore' to exclude a field from struct

* update
Examples: Fix doc typos in accounts.
…s and files (swaggo#642)

* feature: support setting collectionFormat for array param in query

* Use filepath.Dir to take dir of main file so that it runs properly on windows when --parseDependency is set.
Optimize func Skip.

* add test and update README.md

* comment

* fix nil pointer panic in test

* remove print line in test

* feature: exclude dirs and files

* update README.md

* add comment

* Change the implementation of collectionFormat in @param comment

* update README.md
code optimization
code optimization
code optimization
Feat: Support response model composition, post-binding a interface{} field of struct
…ng. (swaggo#655)

* issue 650: fix integer bug in response; support multi fields overriding.

* issue 650: fix integer bug in response; support multi fields overriding.

* report error
necryin and others added 29 commits August 6, 2020 11:28
* parse depth flag

* fix imports

* move to if

Co-authored-by: g.kirilenko <[email protected]>
* Ignore swag file from root dir only

The rule `swag` matches not only file in main directory but also folder in `cmd/swag`.
I've changed it to match only in current folder. For visual I've changed also `swag.exe`.

I've run into issue when I tried to vendor sources for swag but also for tool. The main.go got ignored by git, I was required to `git add -f vendor/github.com/swaggo/swag/cmd/swag/main.go`

* Properly .gitignore from main folder only.
Instead of always sorting parameters with name, which can surprise users
because x-order is ignored, sort them with the slice type declared in
github.com/go-openapi/spec to reuse its comparison method.
--generatedTime was documented as true by default, although immediately
after we stated that it was false by default.

Also capitalized the --exclude docs, and updated README.md's output for
consistency.
update dep latest swaggo/gin-swagger (swaggo#825)
Upgrade github.com/go-openapi/spec and golang.org/x/tools to latest
versions.

Upgrading github.com/go-openapi/spec v0.19.11 -> v.0.19.14 generates
empty "description" members for responses and empty "authorizationUrl"
members for securityDefinitions when they're missing. This causes tests
to fail. Fix by adding the missing empty definitions to expected test
results.
swaggo#837)

* feature: support default response, use comma-separated codes to add a header for multi responses in one line, use keyword 'all' to add a header for all responses

* update readme_zh-CN.md

* fix tests

* fix tests coverage
* issue-503: added type map in example value

* Refactor and update example tag for map(object)

* Add data in test

* Add test

* Add test

Co-authored-by: Eason Lin <[email protected]>
* Remove dependency on gin from tests

Since tests only test parsing of go source and swaggo special comments,
an advanced router is not really necessary

* Clean up modules

* Add inner package to testdata/nested2

Since testdata/nested doesn't point to outside packages anymore, this
caused nested parsing test to skip exercising one if-branch. Add more
nesting to nested2 to regain that.

* Refactor Parser.checkOperationIDUniqueness

Remove duplication in code that checks for duplicate operation ids

* Better duplicate id detection test coverage

Also exercise rest of HTTP methods when testing for duplicate @id
detection
Since example/celler depends on modules that main go.mod doesn't, `go
mod tidy` gets confused. Fix by adding example/celler/go.mod
Added ARM64 architecture to .goreleaser.yml for releasing Linux/ARM64 binary.

Signed-off-by: odidev <[email protected]>

Co-authored-by: Eason Lin <[email protected]>
Update gin-swagger dependency version to mitigate issue with missing dep (github.com/ugorji/go v1.1.5-pre).

Issue link: swaggo#826

Co-authored-by: Eason Lin <[email protected]>
@Nerzal Nerzal merged commit 54bd4bc into master Jan 13, 2021
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.