Skip to content

Commit

Permalink
Merge pull request NixOS#247848 from infinisil/deprecate-lib.types.st…
Browse files Browse the repository at this point in the history
…ring

lib.types.string: Deprecation error instead of warning
  • Loading branch information
roberth authored Aug 8, 2023
2 parents 9331708 + c59c6b1 commit 69d4b01
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions lib/types.nix
Original file line number Diff line number Diff line change
Expand Up @@ -436,10 +436,7 @@ rec {

# Deprecated; should not be used because it quietly concatenates
# strings, which is usually not what you want.
string = separatedString "" // {
name = "string";
deprecationMessage = "See https://github.com/NixOS/nixpkgs/pull/66346 for better alternative types.";
};
string = throw "The type `types.string` is deprecated. See https://github.com/NixOS/nixpkgs/pull/66346 for better alternative types.";

passwdEntry = entryType: addCheck entryType (str: !(hasInfix ":" str || hasInfix "\n" str)) // {
name = "passwdEntry ${entryType.name}";
Expand Down

0 comments on commit 69d4b01

Please sign in to comment.