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.

Sign up for GitHub

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

Jump to bottom

Go sdk v2 initial implementation #16

Merged

pyang55 merged 27 commits into master from go-sdk-v2-initial-implementation

May 6, 2019

Contributor

pyang55 commented May 2, 2019

initial implementation of go sdk v2, all future go sdk v2 features will be branched off of this

pyang55 added 2 commits

May 1, 2019 15:54


          initial go-sdk-v2 implementation

422217d


          fixed collector test

pyang55 requested review from slopeinsb, jesseaukeman and woz5999

May 2, 2019 00:12

ghost added the size/XXL label

pyang55 added 2 commits

May 2, 2019 10:28


          added go-openapi vendor

f32597c


          forgot vendor.json

581c177

pyang55 removed request for slopeinsb, jesseaukeman and woz5999

May 2, 2019 14:47

pyang55 added 3 commits

May 2, 2019 11:01


          added go.mod and go.sum for go-openapi/errors

cf7bf28


          added more items

8f6dafb


          more modules

bfc724a

woz5999 reviewed

View reviewed changes

logicmonitor/config.go Outdated Show resolved Hide resolved

woz5999 reviewed

View reviewed changes

logicmonitor/data_source_collectors.go Show resolved Hide resolved

woz5999 reviewed

View reviewed changes

logicmonitor/provider.go Outdated Show resolved Hide resolved

woz5999 reviewed

View reviewed changes

logicmonitor/resource_collector_group.go Outdated Show resolved Hide resolved

woz5999 reviewed

View reviewed changes

logicmonitor/resource_collector_group.go Outdated Show resolved Hide resolved

woz5999 reviewed

View reviewed changes

logicmonitor/resource_device.go Outdated Show resolved Hide resolved

woz5999 reviewed

View reviewed changes

logicmonitor/resource_device_group.go Outdated Show resolved Hide resolved


          did a go verify

ec4579b

woz5999 suggested changes

View reviewed changes

Contributor

woz5999 left a comment

Left some comments requesting changes.

pyang55 added 7 commits

May 2, 2019 23:25


          testing module input

09e52ae


          did a go tidy

f005ef3


          added vendor items

1e01ca5


          fixed go.sum

8b59e18


          vendor dependencies

be0a716


          one more vendor check

c709bf7


          vendor update

ca77fb5

pyang55 added 11 commits

May 3, 2019 00:05


          vendor update

d069059


          vendor update

358d763


          vendor update

858207d


          vendor update

8af5f02


          more vendor

94757b3


          removed some vendor files

dd9eb2e


          more vendoring

bef1b27


          removed packages

ba01332


          more goodness

20cf745


          final round of vendoring

c2a0790


          added changes to provider based on review suggestions

59d3c03

pyang55 requested a review from woz5999

May 3, 2019 17:01

woz5999 suggested changes

View reviewed changes

logicmonitor/resource_collector_group.go Outdated Show resolved Hide resolved

logicmonitor/resource_device.go Outdated Show resolved Hide resolved

logicmonitor/resource_device_group.go Outdated Show resolved Hide resolved


          changed payload total parameters

ef4427b

woz5999 approved these changes

View reviewed changes

pyang55 merged commit 78c04e9 into master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels