lol

check-meta: filter nulls out of {maintainers,teams}Position (#405141)

authored by winter.bsky.social and committed by

GitHub 4e98cd6b 198786a2

+5 -3
+5 -3
pkgs/stdenv/generic/check-meta.nix
··· 26 26 isAttrs 27 27 isString 28 28 mapAttrs 29 + filterAttrs 29 30 ; 30 31 31 32 inherit (lib.lists) ··· 593 594 ) 594 595 ] ++ optional (hasOutput "man") "man"; 595 596 } 596 - // { 597 - # CI scripts look at these to determine pings. 597 + // (filterAttrs (_: v: v != null) { 598 + # CI scripts look at these to determine pings. Note that we should filter nulls out of this, 599 + # or nix-env complains: https://github.com/NixOS/nix/blob/2.18.8/src/nix-env/nix-env.cc#L963 598 600 maintainersPosition = builtins.unsafeGetAttrPos "maintainers" (attrs.meta or { }); 599 601 teamsPosition = builtins.unsafeGetAttrPos "teams" (attrs.meta or { }); 600 - } 602 + }) 601 603 // attrs.meta or { } 602 604 # Fill `meta.position` to identify the source location of the package. 603 605 // optionalAttrs (pos != null) {