Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@bullgare
Copy link
Contributor

This file can be used for some integrations like Apiary.
It is optional and not turned on by default.
Does not break backward compatibility.

@utrack
Copy link
Owner

utrack commented Jun 16, 2020

LGTM, thanks!

@utrack utrack merged commit 69d768e into utrack:master Jun 16, 2020
@bullgare
Copy link
Contributor Author

Thank you!

@bullgare
Copy link
Contributor Author

@utrack do you have plans to make a release in near future?

@utrack
Copy link
Owner

utrack commented Jun 17, 2020

done, v2.4.9

@bullgare
Copy link
Contributor Author

bullgare commented Jun 17, 2020 via email

@bullgare
Copy link
Contributor Author

filepath.Join(g.options.SwaggerPath, fmt.Sprintf("%s.json", base))

Well, yes, you are right, @NOMORECOFFEE . But we are not about performance here. If we were discussing not merged PR, I'd fix it, but it's been released for about 3 months now.
So if you want it, feel free to create another PR to fix the ordering here.

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.

3 participants