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

Bump github.com/spf13/cobra from 1.1.1 to 1.1.2 #9350

Closed

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Feb 10, 2021

Bumps github.com/spf13/cobra from 1.1.1 to 1.1.2.

Release notes

Sourced from github.com/spf13/cobra's releases.

v1.1.2

Notable Changes

  • Bump license year to 2021 in golden files (#1309) @Bowbaq
  • Enhance PowerShell completion with custom comp (#1208) @Luap99
  • Update gopkg.in/yaml.v2 to v2.4.0: The previous breaking change in yaml.v2 v2.3.0 has been reverted, see go-yaml/yaml#670
  • Documentation readability improvements (#1228 etc.) @zaataylor etc.
  • Use golangci-lint: Repair warnings and errors resulting from linting (#1044) @umarcor
Changelog

Sourced from github.com/spf13/cobra's changelog.

v1.1.2

Notable Changes

  • Bump license year to 2021 in golden files (#1309) @Bowbaq
  • Enhance PowerShell completion with custom comp (#1208) @Luap99
  • Update gopkg.in/yaml.v2 to v2.4.0: The previous breaking change in yaml.v2 v2.3.0 has been reverted, see go-yaml/yaml#670
  • Documentation readability improvements (#1228 etc.) @zaataylor etc.
  • Use golangci-lint: Repair warnings and errors resulting from linting (#1044) @umarcor
Commits
  • 7f95502 Update CHANGELOG.md for v1.1.2
  • 07445ea Copyedit shell-completion related documentation
  • b73b344 ci: add GitHub Actions workflow 'Test' (#1339)
  • 652c755 Use golangci-lint (#1044)
  • 1d71ff0 Deprecate Go < 1.14 (#1323)
  • 1135bde Update gopkg.in/yaml.v2 to v2.4.0
  • 9df156e Cobra User Contract (#1292)
  • ff416ad Revert "Add the ability to specify a filePostpender in GenMarkdownTreeCustom ...
  • 23a6174 Add the ability to specify a filePostpender in GenMarkdownTreeCustom (#1270)
  • 4384b91 Bump license year to 2021 in golden files (#1309)
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [github.com/spf13/cobra](https://github.com/spf13/cobra) from 1.1.1 to 1.1.2.
- [Release notes](https://github.com/spf13/cobra/releases)
- [Changelog](https://github.com/spf13/cobra/blob/master/CHANGELOG.md)
- [Commits](spf13/cobra@v1.1.1...v1.1.2)

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Feb 10, 2021
@helm-bot helm-bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Feb 10, 2021
@marckhouzam
Copy link
Member

This Cobra release may have needed to be a 2.0 because it no longer supports Go < 1.14. This has been brought to the attention of the maintainers (spf13/cobra#1232 (comment)). I'm not sure what the maintainers will do about this.

That being said, the 1.1.2 release should be safe for Helm.
There are three changes worth mentioning:

  1. Cobra no longer supports Go < 1.14
  2. Update gopkg.in/yaml.v2 to v2.4.0: The previous breaking change in yaml.v2 v2.3.0 has been reverted, see Revert "Set the default line length to infinity (-1) (#571)" go-yaml/yaml#670
  3. Full support for PowerShell completion is now available (we may want to offer this to our users)

@mattfarina
Copy link
Collaborator

@marckhouzam Because of the PowerShell completion, is this something for Helm v2.6? I'm surprised to see a feature is in a patch release. Was that intentional?

Go 1.14 isn't supported by the Go community. The latest 2 minor releases is all they support. In the Go community it's not unusual to drop support for older unsupported versions of Go. Other imports we use do it. I'm not suggesting it's right or wrong.... rather that it's normal.

@marckhouzam
Copy link
Member

@marckhouzam Because of the PowerShell completion, is this something for Helm v2.6? I'm surprised to see a feature is in a patch release. Was that intentional?

I hadn't even realized! But yes that's a mistake, if not a 2.0 it should at least have been a 1.2 release.

Go 1.14 isn't supported by the Go community. The latest 2 minor releases is all they support. In the Go community it's not unusual to drop support for older unsupported versions of Go. Other imports we use do it. I'm not suggesting it's right or wrong.... rather that it's normal.

👍

@anthonyfok
Copy link

I will release cobra v1.1.3 with the >= go1.14 requirement reverted. No need for a v1.2 or v2.0 release.

@marckhouzam
Copy link
Member

I will release cobra v1.1.3 with the >= go1.14 requirement reverted. No need for a v1.2 or v2.0 release.

Thanks @anthonyfok taking to time to answer.
The point was that in proper SemVer, there should not be new features in a patch release (1.1.2); new features should be part of a minor release (1.2).

@anthonyfok
Copy link

The point was that in proper SemVer, there should not be new features in a patch release (1.1.2); new features should be part of a minor release (1.2).

AFAIK. there are no new features in the v1.1.2 or v1.1.3 from the v1.1.1 release. PowerShell completion support was already in v1.1.1, and this feature was enhanced in v1.1.2 with no change to API. (Please correct me if I am wrong.)

The artificial bump to go 1.14 in go.mod not for technical reasons but as an "official statement coherent with a User Contract" saying that the Community would not support Go << 1.14... well, that is not what go.mod is for. Nevertheless, thankfully, that change to go.mod did not actually introduce any breaking change either. See my comment for an explanation at spf13/cobra#1232 (comment)

@marckhouzam
Copy link
Member

You are correct that the powershell completion existed in 1.1.1 and has kept the same API in 1.1.3, although it has been greatly improved in its behavior ('greatly' may be a question of opinion😁).

@anthonyfok
Copy link

You are correct that the powershell completion existed in 1.1.1 and has kept the same API in 1.1.3, although it has been greatly improved in its behavior ('greatly' may be a question of opiniongrin).

I totally agree that it is greatly improved too, and that was my main motivation for releasing v1.1.2 to facilitate its inclusion in Debian 11 (entering freeze) and in Hugo.

@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Feb 11, 2021

Superseded by #9355.

@dependabot dependabot bot closed this Feb 11, 2021
@dependabot dependabot bot deleted the dependabot/go_modules/github.com/spf13/cobra-1.1.2 branch February 11, 2021 05:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants