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

Update and remove deps #265

Merged
merged 3 commits into from
Jun 14, 2021
Merged

Update and remove deps #265

merged 3 commits into from
Jun 14, 2021

Conversation

tvoran
Copy link
Member

@tvoran tvoran commented Jun 10, 2021

Updates yaml.v2 and golang.org/x/crypto. Removes the dependency
on consul by adding a simple version of consul's flags.Usage()
for printing command options for agent-inject.

tvoran added 3 commits June 9, 2021 11:45
Added a simple version of consul's flags.Usage() for printing command
options for agent-inject, so that the consul dependency could be
removed.
@tvoran tvoran requested review from jasonodonnell and tomhjp June 10, 2021 20:42
@calvn
Copy link
Contributor

calvn commented Jun 14, 2021

Does go mod tidy remove any of the indirect dependencies?

@tvoran
Copy link
Member Author

tvoran commented Jun 14, 2021

@calvn It doesn't seem to for me.

@tvoran tvoran merged commit 9aae5b8 into master Jun 14, 2021
@tvoran tvoran deleted the VAULT-2572/update-deps branch June 14, 2021 19:05
RemcoBuddelmeijer pushed a commit to RemcoBuddelmeijer/vault-k8s that referenced this pull request Feb 22, 2022
* update golang.org/x/crypto

* update yaml.v2

* remove consul dep

Added a simple version of consul's flags.Usage() for printing command
options for agent-inject, so that the consul dependency could be
removed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants