Skip to content

lockFlake(): When updating a lock, respect the input's lock file (backport #13437)#13439

Merged
tomberek merged 1 commit into2.30-maintenancefrom
mergify/bp/2.30-maintenance/pr-13437
Jul 10, 2025
Merged

lockFlake(): When updating a lock, respect the input's lock file (backport #13437)#13439
tomberek merged 1 commit into2.30-maintenancefrom
mergify/bp/2.30-maintenance/pr-13437

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Jul 10, 2025

Motivation

For example, if a flake has a lock for a and a/b, and we change the flakeref for a, previously Nix would fetch the latest version of b rather than using the lock for b from a.

Fixes #8976 and probably #6105. Updated version of #8977.

Context


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.


This is an automatic backport of pull request #13437 done by [Mergify](https://mergify.com).

@mergify mergify bot requested a review from edolstra as a code owner July 10, 2025 04:29
@mergify mergify bot added automatic backport This PR is a backport produced by automation (does not trigger backporting) merge-queue labels Jul 10, 2025
@github-actions github-actions bot added the with-tests Issues related to testing. PRs with tests have some priority label Jul 10, 2025
@tomberek tomberek enabled auto-merge July 10, 2025 04:34
@mergify
Copy link
Copy Markdown
Contributor Author

mergify bot commented Jul 10, 2025

This pull request has been removed from the queue for the following reason: pull request dequeued.

Pull request #13439 has been dequeued. The pull request could not be merged. This could be related to an activated branch protection or ruleset rule that prevents us from merging. (details: Required status check "tests on ubuntu" is in progress.).

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.
If you do update this pull request, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio requeue comment.

@tomberek tomberek merged commit f880135 into 2.30-maintenance Jul 10, 2025
25 of 27 checks passed
@tomberek tomberek deleted the mergify/bp/2.30-maintenance/pr-13437 branch July 10, 2025 05:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automatic backport This PR is a backport produced by automation (does not trigger backporting) merge-queue with-tests Issues related to testing. PRs with tests have some priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants