X.P.Eval reexport defaultEvalConfig#37
Conversation
|
This can't be merged until CI is fixed (#36 which is still WIP). |
geekosaur
force-pushed
the
reexport-eval-config
branch
from
b2b3fa1 to
0ddd6ad
Compare
|
Gaaah. Something up with Ubuntu repos or GHA's ability to access them? |
geekosaur
force-pushed
the
reexport-eval-config
branch
from
0ddd6ad to
1635b4d
Compare
|
So, while the commit to fix CI passed as a PR, somehow after commit it became necessary to add a |
Probably just a timing issue. Whatever Ubuntu image it's running, it has apt lists from a few days/weeks ago so it might work for a bit and then stop. The workflows in xmonad have had apt update for aaaages |
It's in `X.A.Eval`, but the documentation doesn't tell you that you need to import it from there. And really, you shouldn't have to; it should be reexported. (Yes, I know we've gone all `def`, but I still consider it a very bad idea especially after it caused breakage with the hide/show docks key.)
geekosaur
force-pushed
the
reexport-eval-config
branch
from
1635b4d to
a767d09
Compare
|
FTR I ended up slipping it into the other PR, which has an approval and therefore got merged first. Rebasing caused it to disappear here. |
3 participants
It's in
X.A.Eval, but the documentation doesn't tell you that you need to import it from there. And really, you shouldn't have to; it should be reexported.(Yes, I know we've gone all
def, but I still consider it a very bad idea especially after it caused breakage with the hide/show docks key.)