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

[skip changelog] Update cobra to latest version #1430

Merged
merged 4 commits into from
Sep 3, 2021
Merged

Conversation

umbynos
Copy link
Contributor

@umbynos umbynos commented Sep 2, 2021

Please check if the PR fulfills these requirements

  • The PR has no duplicates (please search among the Pull Requests
    before creating one)
  • The PR follows
    our contributing guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • UPGRADING.md has been updated with a migration guide (for breaking changes)
  • What kind of change does this PR introduce?

Dependency update

  • What is the current behavior?
  • What is the new behavior?

This PR is required to add the newly added dynamic completion 😏
I also updated the tests since the completion for powershell is changed a little bit

no

  • Other information:

See how to contribute

@umbynos umbynos requested a review from a team September 2, 2021 14:57
@umbynos umbynos self-assigned this Sep 2, 2021
@silvanocerza silvanocerza changed the title Update cobra to latest version [skip changelog] Update cobra to latest version Sep 3, 2021
@umbynos umbynos merged commit 9133101 into master Sep 3, 2021
@umbynos umbynos deleted the umbynos/bump_cobra branch September 3, 2021 08:37
@per1234 per1234 added topic: infrastructure Related to project infrastructure and removed topic: dependencies labels Sep 24, 2021
silvanocerza pushed a commit that referenced this pull request Sep 30, 2021
* bump github.com/spf13/cobra to 1.2.1

* go mod tidy

* fix CI failing

* update powershell tests, in new version of cobra the completion file is different
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: infrastructure Related to project infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants