do not force set the api logger to log.Printf #1962
Merged
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
Codecov Report
@@ Coverage Diff @@
## master #1962 +/- ##
==========================================
+ Coverage 80.61% 80.63% +0.02%
==========================================
Files 38 38
Lines 7639 7639
==========================================
+ Hits 6158 6160 +2
+ Misses 991 990 -1
+ Partials 490 489 -1
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #1962 +/- ##
==========================================
- Coverage 80.91% 80.63% -0.29%
==========================================
Files 47 38 -9
Lines 8066 7639 -427
==========================================
- Hits 6527 6160 -367
+ Misses 1043 990 -53
+ Partials 496 489 -7
Continue to review full report at Codecov.
|
fredbi
requested changes
Jun 6, 2019
djgilcrease
force-pushed
the
master
branch
2 times, most recently
from
c7c9b4f to
e3f71b8
Compare
fredbi
approved these changes
Jun 6, 2019
|
@djgilcrease This one just needs an update with regenerated bindata |
Adds a badge showing the number of people helping this repo on CodeTriage. [](https://www.codetriage.com/go-swagger/go-swagger) ## What is CodeTriage? CodeTriage is an Open Source app that is designed to make contributing to Open Source projects easier. It works by sending subscribers a few open issues in their inbox. If subscribers get busy, there is an algorithm that backs off issue load so they do not get overwhelmed [Read more about the CodeTriage project](https://www.codetriage.com/what). ## Why am I getting this PR? Your project was picked by the human, @schneems. They selected it from the projects submitted to https://www.codetriage.com and hand edited the PR. How did your project get added to [CodeTriage](https://www.codetriage.com/what)? Roughly over 3 years ago, [tamarabartlett](https://github.com/tamarabartlett) added this project to CodeTriage in order to start contributing. Since then, 13 people have subscribed to help this repo. ## What does adding a badge accomplish? Adding a badge invites people to help contribute to your project. It also lets developers know that others are invested in the longterm success and maintainability of the project. You can see an example of a CodeTriage badge on these popular OSS READMEs: - [](https://www.codetriage.com/rails/rails) https://github.com/rails/rails - [](https://www.codetriage.com/crystal-lang/crystal) https://github.com/crystal-lang/crystal ## Have a question or comment? While I am a bot, this PR was manually reviewed and monitored by a human - @schneems. My job is writing commit messages and handling PR logistics. If you have any questions, you can reply back to this PR and they will be answered by @schneems. If you do not want a badge right now, no worries, close the PR, you will not hear from me again. Thanks for making your project Open Source! Any feedback is greatly appreciated. Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
the logger already defaults so do not force set it in `SetAPI` since that would overwrite one I set in the API before calling `SetAPI` Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
Use the correct signature for the basic authentication interface. This caused the compilation to fail. Fixes go-swagger#1765 Signed-off-by: Joakim Soderberg <joakim.soderberg@gmail.com> Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
as for only-one-tag operations, instead of default "operations" as when no tags are available. Signed-off-by: Maxime Soulé <btik-git@scoubidou.com> Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
Signed-off-by: kofler <efi.kofler@flir.com> Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
* x-go-type: if no imports are specified, assume that the referred type is inside the same package Signed-off-by: Emil Nikolov <emil.e.nikolov@gmail.com> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * support keep spec properties order Signed-off-by: Efi-Kofler <efi.kofler@tectonic.network> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Added diff sub command Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Refactored location node code. Detected reponse and param header changes Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Updated test fixtures Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Removed redundant method and fixed default/required on response headers Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * urlbuilder template fixed to support query params named 'append' (go-swagger#1949) fixes go-swagger#1948 Update urlbuilder_test.go to support added query name mangling. Update bindata.go Signed-off-by: Jim Turnquist <jim@boonlogic.com> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Refactored type names and removed some dead code Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Refactored analyser and diff code in prep for ignore processing Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Added ability to igore selected diffs Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Added ability to ignore deprecated operations Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Added docs Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Fixed typo compatabilty -> compatibilty Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Fixed typo and scopelint issue Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * scan: proper parse for numeric elements (future validation pass). Signed-off-by: Dzmitry.Harupa <dzmitry.harupa@1pt.com> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * fix test Signed-off-by: Dzmitry.Harupa <dzmitry.harupa@1pt.com> fix building Signed-off-by: Dzmitry.Harupa <dzmitry.harupa@1pt.com> review: goconst Signed-off-by: d7561985 <d7561985@gmail.com> update Signed-off-by: d7561985 <d7561985@gmail.com> test fix Signed-off-by: d7561985 <d7561985@gmail.com> test fix Signed-off-by: d7561985 <d7561985@gmail.com> add const for linter Signed-off-by: d7561985 <d7561985@gmail.com> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Added gocrest in various spots Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Updating vendor folder and adding coverage.txt to gitignore Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Lower-cased error - lint warning Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Updated go.mod Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Added files nuked by rogue gitignore Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Added files lost due to bad gitignore Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Updated readme Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Updated README Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Adding lost code and new tests Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Updated ignore diff test Signed-off-by: Jason Wraxall <> Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Circle CI closed some issues Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Updated lint settings as recommended by reviewer Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Updated capitalisation of global Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * ran go gmt as asked Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * fixed loop scope issue Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * fixed more formatting Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * fixed yet more formatting Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * ran goimports Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * nuked spurious warning from dupl Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Removed dead code and added an array diff test Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Resolved code suggestion Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Cleaning code Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * moved test code into diff package to improve coverage report Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * fixed lint suggestion Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Added kitchensink to diff fictures to improve coverage and fixed the bugs the new test cases detected ;-) Signed-off-by: Jason Wraxall <wjase@yahoo.com.au> * Improving formatting and coverage at the same time Signed-off-by: Jason Wraxall <jwraxall@tyro.com> * Fixed cicd issues Signed-off-by: Jason Wraxall <jwraxall@tyro.com> * Added test for map diff Signed-off-by: Jason Wraxall <jwraxall@tyro.com> * Removed dead code Signed-off-by: Jason Wraxall <jwraxall@tyro.com> * Added empty arg error test Signed-off-by: Jason Wraxall <jwraxall@tyro.com> * Added test cases to get above 80% coverage Signed-off-by: Jason Wraxall <jwraxall@tyro.com> * Added extra test permutations to boost coverage Signed-off-by: Jason Wraxall <jwraxall@tyro.com> Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
…ndata.go from coverage Signed-off-by: Frederic BIDON <fredbi@yahoo.com> Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
…nterface{})
fixes go-swagger#1490
Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
* fixes go-swagger#973 Signed-off-by: Frederic BIDON <fredbi@yahoo.com> Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
fixes go-swagger#1020 Signed-off-by: Frederic BIDON <fredbi@yahoo.com> Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
Fixes go-swagger#1339 Signed-off-by: Frederic BIDON <fredbi@yahoo.com> Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
Signed-off-by: Frederic BIDON <fredbi@yahoo.com> Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
Signed-off-by: Frederic BIDON <fredbi@yahoo.com> Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
* fixes go-swagger#1893 * codegen (client and server) now supports codes other than the well-known ones published by go-openapi/runtime Signed-off-by: Frederic BIDON <fredbi@yahoo.com> Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
… default response with error rather than panicking (go-swagger#1978) Fixes go-swagger#1518 Signed-off-by: Frederic BIDON <fredbi@yahoo.com> Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
Signed-off-by: wych42 <wych101010@gmail.com> Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
Signed-off-by: Frederic BIDON <fredbi@yahoo.com> Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
the logger already defaults so do not force set it in `SetAPI` since that would overwrite one I set in the API before calling `SetAPI` Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
Signed-off-by: Dj Gilcrease <d.gilcrease@f5.com>
](https://www.codetriage.com/go-swagger/go-swagger){kind=link}
](https://www.codetriage.com/rails/rails){kind=link}
](https://www.codetriage.com/crystal-lang/crystal){kind=link}
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.
the logger already defaults so do not force set it in
SetAPIsince that would overwrite one I set in the API before callingSetAPI