Skip to content

Conversation

@dependabot
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github May 12, 2025

Bumps github.com/mikefarah/yq/v4 from 4.45.2 to 4.45.4.

Release notes

Sourced from github.com/mikefarah/yq/v4's releases.

v4.45.4 - Fixing wrong map() behaviour on empty map

  • Fixing wrong map() behaviour on empty map #2359
  • Bumped dependencies

v4.45.3 - Fixes regression bug(s)

  • Fixing regression (#2353, #2359, #2325) introduced with in 4.45.2 with #2325 fix
  • Bumped dependencies

Sorry for the regression folks! 😓 fwiw I have since added automated tests to capture the scenarios provided in the regression bug tickets

Changelog

Sourced from github.com/mikefarah/yq/v4's changelog.

4.45.4:

  • Fixing wrong map() behaviour on empty map #2359
  • Bumped dependencies

4.45.3:

  • Fixing regression introduced with in 4.45.2 with #2325 fix 😓 sorry folks!
  • Bumped dependencies
Commits
  • b534aa9 Bumping version
  • 39a65b6 Updating release notes
  • 1e3006e Removing old issue template
  • 22949df Fixing running map against empty array bug #2359
  • 734e2cd Bumping version
  • 082b76a Preparing release notes
  • 5bc2cd0 Bumping golang version
  • 20407a0 Adding more tests to prevent regression again :sweat
  • 0a83da6 Bump github.com/pelletier/go-toml/v2 from 2.2.3 to 2.2.4
  • 77da8b7 Bump golang from 1.24.2 to 1.24.3
  • 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 show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @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)

@dependabot dependabot bot added dependencies Pull requests that update a dependency file go Pull requests that update Go code labels May 12, 2025
@AkihiroSuda AkihiroSuda modified the milestone: v1.1.0 May 12, 2025
module github.com/lima-vm/lima

go 1.23.0
go 1.24
Copy link
Member

Choose a reason for hiding this comment

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

Not sure if that is acceptable for us. It looks like a gratuitous bump; should I ask upstream why they changed it?

Copy link
Member

Choose a reason for hiding this comment

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

Let's keep 1.23 unless this update is urgent

Copy link
Member

Choose a reason for hiding this comment

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

We should revert #3494 then and stay with yq 4.45.1. The 4.45.3 and 4.45.4 releases are fixing regressions introduced in 4.45.2, which we bumped to in #3494.

I've also filed mikefarah/yq#2367 to ask why yq requires Go 1.24 (I don't think it actually does).

Copy link
Member

Choose a reason for hiding this comment

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

Revert in #3535

Copy link
Member

Choose a reason for hiding this comment

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

And as far as I know, you even have to depend on "1.24.0" now (since go1.21). None of this fuzzy/friendly

Copy link
Member

Choose a reason for hiding this comment

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

I've now taken a closer look, and the requirement for 1.24 in yq does come from one of its dependencies: a8m/envsubst#66

This still looks like a misguided use of the go line to force the user to upgrade Go, but I don't think there is any point in following up on this.

I think we should stay with 1.23 until Lima 1.10 is out (and keep yq at 4.45.1 until then), and then consider moving to Go 1.24 afterwards.

Copy link
Member

Choose a reason for hiding this comment

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

This still looks like a misguided use

You can see the misunderstanding by how in CI they (a8m/envsubst) still test compiling with both 1.23 and 1.24.

This makes no sense because the go 1.24 line will ensure that both jobs are compiled with 1.24.

@jandubois jandubois marked this pull request as draft May 12, 2025 16:21
Bumps [github.com/mikefarah/yq/v4](https://github.com/mikefarah/yq) from 4.45.2 to 4.45.4.
- [Release notes](https://github.com/mikefarah/yq/releases)
- [Changelog](https://github.com/mikefarah/yq/blob/master/release_notes.txt)
- [Commits](mikefarah/yq@v4.45.2...v4.45.4)

---
updated-dependencies:
- dependency-name: github.com/mikefarah/yq/v4
  dependency-version: 4.45.4
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot force-pushed the dependabot/go_modules/github.com/mikefarah/yq/v4-4.45.4 branch from 5657235 to 18dc64b Compare May 12, 2025 23:48
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github May 12, 2025

OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting @dependabot ignore this major version or @dependabot ignore this minor version. You can also ignore all major, minor, or patch releases for a dependency by adding an ignore condition with the desired update_types to your config file.

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/mikefarah/yq/v4-4.45.4 branch May 12, 2025 23:49
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 go Pull requests that update Go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants