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

Lint fix #1504

Merged
merged 6 commits into from
Oct 26, 2019
Merged

Lint fix #1504

merged 6 commits into from
Oct 26, 2019

Conversation

sriv
Copy link
Member

@sriv sriv commented Oct 17, 2019

fixed issues reported by govet, errcheck, gosimple etc. run golangci-lint run.

details: https://github.com/golangci/golangci-lint

@gaugebot gaugebot bot added the cla-signed label Oct 17, 2019
@gaugebot gaugebot bot requested a review from shubhamsc October 17, 2019 03:04
@shubhamsc shubhamsc merged commit 03fd17f into master Oct 26, 2019
@shubhamsc shubhamsc deleted the lint_fix branch October 26, 2019 14:13
Apoorva-GA pushed a commit that referenced this pull request Oct 31, 2019
* Fix golangci issues: 
* Remove unused fields
* Added error checks
negiDharmendra pushed a commit that referenced this pull request Nov 5, 2019
…#1490 (#1511)

* Removing all telemetry related code and deprecating the telemetry commands #1490

* Fixing deep source errors #1490

* Fixing deep source error #1490

* Removing unique id generation #1490

* Removing useTestGA env var #1490

* Lint fix (#1504)

* Fix golangci issues: 
* Remove unused fields
* Added error checks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants