add mongodb #28
add mongodb #28
Conversation
|
All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
LICENSE
Outdated
There was a problem hiding this comment.
add license for package appearing in go.mod, eg: " mongo-driver"
MIT: goconvey, gomonkey/v2, testify
Apache-2.0： mock, mongo-driver, yaml.v3
There was a problem hiding this comment.
add mongo-driver in LICENSE
mongodb/curd.go
Outdated
|
|
||
| // MongoArgFilter mongo operation instruction | ||
| const ( | ||
| MongoArgFilter = "filter" |
There was a problem hiding this comment.
These constants do not need to be exported
There was a problem hiding this comment.
modify these constants to be non-exported
mongodb/codec.go
Outdated
|
|
||
| // default codec | ||
| var ( | ||
| DefaultClientCodec = &ClientCodec{} |
There was a problem hiding this comment.
do not need to be exported
mongodb/plugin.go
Outdated
|
|
||
| // MongoPlugin is used for plug-in default initialization, | ||
| // used to load mongo proxy connection parameter configuration. | ||
| type MongoPlugin struct{} |
There was a problem hiding this comment.
do not need to be exported
|
Please add some PR description |
@sandyskies done |
Updated the relevant dependencies and LICENSE.
Open source mongodb to GitHub trpc ecosystem.