tyk icon indicating copy to clipboard operation
tyk copied to clipboard

[TT-5956] proto: remove unused import and clear latest protoc warning

Open matiasinsaurralde opened this issue 3 years ago • 5 comments

Description

Related Issue

Motivation and Context

How This Has Been Tested

Screenshots (if appropriate)

Types of changes

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to change)
  • [ ] Refactoring or add test (improvements in base code or adds test coverage to functionality)

Checklist

  • [ ] Make sure you are requesting to pull a topic/feature/bugfix branch (right side). If pulling from your own fork, don't request your master!
  • [ ] Make sure you are making a pull request against the master branch (left side). Also, you should start your branch off our latest master.
  • [ ] My change requires a change to the documentation.
    • [ ] If you've changed APIs, describe what needs to be updated in the documentation.
    • [ ] If new config option added, ensure that it can be set via ENV variable
  • [ ] I have updated the documentation accordingly.
  • [ ] Modules and vendor dependencies have been updated; run go mod tidy && go mod vendor
  • [ ] When updating library version must provide reason/explanation for this update.
  • [ ] I have added tests to cover my changes.
  • [ ] All new and existing tests passed.
  • [ ] Check your code additions will not fail linting checks:
    • [ ] go fmt -s
    • [ ] go vet

matiasinsaurralde avatar Jul 13 '22 19:07 matiasinsaurralde

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

sonarqubecloud[bot] avatar Jul 13 '22 19:07 sonarqubecloud[bot]

API tests result: success :white_check_mark: Branch used: refs/pull/4183/merge Commit:
Triggered by: pull_request (@matiasinsaurralde) Execution page

Tyk-ITS avatar Jul 13 '22 19:07 Tyk-ITS

API tests result: success :white_check_mark: Branch used: refs/pull/4183/merge Commit: 248bf17a10f433dbd9dc05246242280dc1ecdc3b Triggered by: pull_request (@ermirizio) Execution page

Tyk-ITS avatar Sep 08 '22 22:09 Tyk-ITS

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

sonarqubecloud[bot] avatar Sep 15 '22 15:09 sonarqubecloud[bot]

API tests result: failure :no_entry_sign: Branch used: refs/pull/4183/merge Commit: 659cde24e2d1362aca760e5c67e737ab4ebc08de Triggered by: pull_request (@tbuchaillot) Execution page

buger avatar Sep 15 '22 15:09 buger

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

sonarqubecloud[bot] avatar Sep 27 '22 17:09 sonarqubecloud[bot]

API tests result: success :white_check_mark: Branch used: refs/pull/4183/merge Commit: cfb606c87c4a14616cf76b040493ff501aaddaea Triggered by: pull_request (@buger) Execution page

Tyk-ITS avatar Sep 27 '22 17:09 Tyk-ITS