Skip to content
This repository has been archived by the owner on Jul 25, 2022. It is now read-only.

refactor curl to use net/http #274

Merged
merged 1 commit into from
Aug 27, 2020

Conversation

neo-liang-sap
Copy link
Contributor

@neo-liang-sap neo-liang-sap commented Aug 25, 2020

What this PR does / why we need it:
refactor curl to use net/http
Which issue(s) this PR fixes:
Fixes #273

Special notes for your reviewer:

Release note:

refactor curl to use net/http

@neo-liang-sap neo-liang-sap requested a review from a team as a code owner August 25, 2020 03:01
@gardener-robot-ci-2 gardener-robot-ci-2 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Aug 25, 2020
@gardener-robot-ci-3 gardener-robot-ci-3 added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Aug 25, 2020
@gardener-robot-ci-3 gardener-robot-ci-3 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Aug 25, 2020
@gardener-robot-ci-2 gardener-robot-ci-2 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Aug 25, 2020
@tedteng
Copy link
Contributor

tedteng commented Aug 25, 2020

/lgtm

@gardener-robot gardener-robot added the reviewed/lgtm Has approval for merging label Aug 25, 2020
pkg/cmd/miscellaneous.go Outdated Show resolved Hide resolved
@gardener-robot-ci-2 gardener-robot-ci-2 added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Aug 26, 2020
@neo-liang-sap neo-liang-sap mentioned this pull request Aug 26, 2020
@petersutter
Copy link
Contributor

Please do not amend your commits and force push. Instead just commit every change you do so that I can better keep track what is new and what you have changed, thanks.

Once we are done with the review you can then squash it finally.

@neo-liang-sap
Copy link
Contributor Author

Please do not amend your commits and force push. Instead just commit every change you do so that I can better keep track what is new and what you have changed, thanks.

Once we are done with the review you can then squash it finally.

Sorry about that, will note this point next time, thanks for reminding!

@gardener-robot-ci-3 gardener-robot-ci-3 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Aug 26, 2020
@gardener-robot-ci-1 gardener-robot-ci-1 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Aug 26, 2020
Copy link
Contributor

@petersutter petersutter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

I haven't tested the code but it lgtm, thanks

@gardener-robot-ci-2 gardener-robot-ci-2 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Aug 27, 2020
@gardener-robot-ci-1 gardener-robot-ci-1 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Aug 27, 2020
@neo-liang-sap neo-liang-sap merged commit 965936e into gardener-attic:master Aug 27, 2020
@neo-liang-sap neo-liang-sap deleted the curl-to-api branch August 27, 2020 05:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) reviewed/lgtm Has approval for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refactor ExecCmdReturnOutput usage of curl to use API access
7 participants