Dhall over Pkl? #682
Replies: 4 comments 8 replies
-
|
The "I am no longer maintaining this project" doesn't inspire a lot of confidence 🙃 |
Beta Was this translation helpful? Give feedback.
-
|
But it is a complete implementation of an earlier Dhall spec, according to their testsuite. If someone (me) took it over, would you consider it?
▰▰▰▰▰
Miles Wirht 🙃
… From: Josh Cannon ***@***.***>
Sent: 05 February 2026 17:27
To: jdx/hk ***@***.***>
Cc: Miles Wirht ***@***.***>, Author ***@***.***>
Subject: Re: [jdx/hk] Dhall over Pkl? (Discussion #682)
The "I am no longer maintaining this project" doesn't inspire a lot of confidence 🙃
--
Reply to this email directly or view it on GitHub:
#682 (comment)
You are receiving this because you authored the thread.
Message ID: ***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
|
I'm not the maintainer, but I think the answer would almost certainly be "no". It'd be a disruptive change for little gain, and has some serious supply chain security issues. |
Beta Was this translation helpful? Give feedback.
-
|
Can't fathom what supply chain securities arise out of reducing attack surface? Right now there's two binaries to trust (Pkl and hk). |
Beta Was this translation helpful? Give feedback.
-
|
Pkl is produced by Apple, which is a trustworthy organization (all things considered). Dhall is produced by...? And the rust crate which is currently unmaintained would then be maintained by one person. |
Beta Was this translation helpful? Give feedback.
-
These are trust concerns, not supply chain concerns. https://dhall-lang.org/ is more legit than most of the dependencies in the project now, with a backing foundation. |
Beta Was this translation helpful? Give feedback.
-
|
Trust is security |
Beta Was this translation helpful? Give feedback.
-
|
I think dhall looks like vomit |
Beta Was this translation helpful? Give feedback.
-
|
See response |
Beta Was this translation helpful? Give feedback.
-
|
I think that's entirely because of the leading commas, see below:
```dhall
let Config = ./pkl/Config.dhall
let Builtins = ./pkl/Builtins.dhall
let Step = Config.Step
let Group = Config.Group
let linters
: List { mapKey : Text, mapValue : Step }
= [
{
mapKey = "actionlint",
mapValue = Builtins.actionlint
},
{
mapKey = "cargo-fmt",
mapValue = Builtins.cargo_fmt
},
{
mapKey = "cargo-clippy",
mapValue =
Builtins.cargo_clippy
// {
profiles = Some [ "slow" ],
check =
Some
"cargo clippy --manifest-path {{workspace_indicator}} --quiet -- -D warnings"
}
},
{
mapKey = "cargo-check",
mapValue = Builtins.cargo_check // { profiles = Some [ "!slow" ] }
},
{
mapKey = "dbg",
mapValue =
{
glob = "**/*.rs",
check = "! rg -e 'dbg!' {{files}}"
}
},
{
mapKey = "eslint",
mapValue =
Builtins.eslint // { dir = Some "docs", prefix = Some "bun run" }
},
{
mapKey = "newlines",
mapValue = Builtins.newlines
},
{
mapKey = "trailing-whitespace",
mapValue = Builtins.trailing_whitespace
},
{
mapKey = "prettier",
mapValue =
Builtins.prettier
// { glob = Some [ "*.js", "*.ts", "*.yml", "*.yaml" ] }
},
{
mapKey = "pkl-format",
mapValue = Builtins.pkl_format
},
{
mapKey = "taplo",
mapValue = Builtins.taplo // { glob = Some "settings.toml" }
},
{
mapKey = "taplo-format",
mapValue = Builtins.taplo_format
},
{
mapKey = "docs",
mapValue =
{
glob = "docs/**",
check = "mise run docs:build",
output_summary = Some "hide"
}
}
]
in Config
// {
hooks =
toMap
{
pre-commit =
{
fix = Some True,
stash = Some "git",
steps =
[
{
mapKey = "prelint",
mapValue =
{
check = "mise run prelint",
exclusive = Some True
}
}
]
# linters
# [
{
mapKey = "postlint",
mapValue =
{
check = "echo postlint",
exclusive = Some True
}
}
]
},
pre-push = { steps = linters },
check = { steps = linters },
fix = { fix = Some True, steps = linters }
},
warnings = Some [ "missing-profiles" ]
}
```
But it's preference. I won't push it.
▰▰▰▰▰
Miles Wirht 🙃
… From: jdx ***@***.***>
Sent: 06 February 2026 05:23
To: jdx/hk ***@***.***>
Cc: Miles Wirht ***@***.***>, Author ***@***.***>
Subject: Re: [jdx/hk] Dhall over Pkl? (Discussion #682)
I think dhall looks like vomit
--
Reply to this email directly or view it on GitHub:
#682 (comment)
You are receiving this because you authored the thread.
Message ID: ***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
|
(FYI the formatting isn't right) |
Beta Was this translation helpful? Give feedback.
-
|
am aware and yet it works in the github preview before publishing |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
Shares all of the same programmable qualities, and yet has a in-rust parsing library, so no CLI-dependencies.
Beta Was this translation helpful? Give feedback.
All reactions