Skip to content

build(deps): Bump github.com/spf13/cobra from 1.1.1 to 1.1.3#161

Closed
dependabot[bot] wants to merge 1 commit intomasterfrom
dependabot/go_modules/github.com/spf13/cobra-1.1.3
Closed

build(deps): Bump github.com/spf13/cobra from 1.1.1 to 1.1.3#161
dependabot[bot] wants to merge 1 commit intomasterfrom
dependabot/go_modules/github.com/spf13/cobra-1.1.3

Conversation

@dependabot
Copy link
Contributor

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

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

Release notes

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

v1.1.3

  • Fix: release-branch.cobra1.1 only: Revert "Deprecate Go < 1.14" to maintain backward compatibility

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.3

  • Fix: release-branch.cobra1.1 only: Revert "Deprecate Go < 1.14" to maintain backward compatibility

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

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.3.
- [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.3)

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added the T:dependencies Type: Pull requests that update a dependency file label Feb 11, 2021
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Feb 11, 2021

Dependabot tried to add @melekes, @tessr and @erikgrinaker as reviewers to this PR, but received the following error from GitHub:

POST https://api.github.com/repos/lazyledger/lazyledger-core/pulls/161/requested_reviewers: 422 - Reviews may only be requested from collaborators. One or more of the users or teams you specified is not a collaborator of the lazyledger/lazyledger-core repository. // See: https://docs.github.com/rest/reference/pulls#request-reviewers-for-a-pull-request

@liamsi
Copy link
Member

liamsi commented Feb 18, 2021

The SDK up to version 0.41.x still uses cobra 1.1.1. Closing this as I'd like to keep version parity between ll-core and the latest SDK for now.

@liamsi liamsi closed this Feb 18, 2021
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Feb 18, 2021

OK, I won't notify you again about this release, but will get in touch when a new version is available.

If you change your mind, just re-open this PR and I'll resolve any conflicts on it.

@dependabot dependabot bot deleted the dependabot/go_modules/github.com/spf13/cobra-1.1.3 branch February 18, 2021 21:12
evan-forbes pushed a commit that referenced this pull request Mar 11, 2025
Follow-up to #137.

Fixes the Docker builds that were failing to be parsed by GitHub Actions.

Also fixes our `make build-docker` target to build images in the same way we do in CI. On Linux, I have to explicitly tell Docker to use BuildKit to pass the target/build platform args in:

```bash
DOCKER_BUILDKIT=1 make build-docker
```

Given that I've also proven now that the images do build in CI (see [this workflow](https://github.com/cometbft/cometbft/actions/runs/3964436268/jobs/6793294390) and [this one](https://github.com/cometbft/cometbft/actions/runs/3964436273/jobs/6793294238)), we no longer need to build on every change to every pull request. The E2E node Docker image in particular takes nearly 30mins to build, so that would slow us down tremendously. It's far better to build those images only once PRs get merged.

---

#### PR checklist

- [x] Tests written/updated, or no tests needed
- [x] `CHANGELOG_PENDING.md` updated, or no changelog entry needed
- [x] Updated relevant documentation (`docs/`) and code comments, or no
      documentation updates needed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T:dependencies Type: Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant