Change types.attrs to recursively merge#76760
Conversation
|
I'm not a fan of this |
|
Thank you for your contributions. This has been automatically marked as stale because it has had no activity for 180 days. If this is still important to you, we ask that you leave a comment below. Your comment can be as simple as "still important to me". This lets people see that at least one person still cares about this. Someone will have to do this at most twice a year if there is no other activity. Here are suggestions that might help resolve this more quickly:
|
|
I marked this as stale due to inactivity. → More info |
|
We have the settings options that can do this. And many services are migrated to this already. |
Motivation for this change
This is a simple change to make
types.attrsuse recurisve update merge strategy. I currently cannot think of anything that this change may break, but not sure if I missed anything. So just putting this out for discussion.Fixes #32217
Things done
sandboxinnix.confon non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"./result/bin/)nix path-info -Sbefore and after)