flatMapAttrs is easier to read because it introduces the values before using them, kind of like a let bindings with multiple values.
The repeated comments remind the reader of the purpose of the innermost attrsets, which is actually very simple. Knowing that they go right into the result should help a lot with building a mental model for this pattern.
Context
flatMapAttrs was added in #10955 and has some docs of its own.
Motivation
Make it easier to read.
flatMapAttrsis easier to read because it introduces the values before using them, kind of like aletbindings with multiple values.The repeated comments remind the reader of the purpose of the innermost attrsets, which is actually very simple. Knowing that they go right into the result should help a lot with building a mental model for this pattern.
Context
flatMapAttrswas added in #10955 and has some docs of its own.Priorities and Process
Add :+1: to pull requests you find important.
The Nix maintainer team uses a GitHub project board to schedule and track reviews.