nickstenn: I'm going to take a look at the CLI right now
but I might get distracted in a second
overall: +1
details: argparse++ (clik was a bit silly)
nickstenn
tilgovi: yeah. I haven't removed clik yet, but it's a lot easier to see that it is removable now.
tilgovi
concerns: your comments suggest that h.script _doesn't_ use paster.bootstrap but it does. I would just make sure we're on a trajectory to consolidate these modules and remove the old stuff
oh, wait, maybe it doesn't
it uses get_appsettings, though
so it is reading the same config file, obviously
nickstenn
tilgovi: it didn't actually use bootstrap
the issue was just the crapshoot of manually doing config.include
so when things move inside the application, the CLI broke
now if the application boots, the commands work
vannevar has quit
(hopefully)
tilgovi
hm
I wonder why I had to do that
vannevar joined the channel
ohh
because even with get_appsettings, the main app is not in pyramid.includes
it's the use=
nickstenn
right
anyway, I think it's cleaner now
tilgovi
cool
I'll read it over momentarily
nickstenn
tilgovi: can I kill hypothesis/insight?
shepazu joined the channel
shepazu has quit
kfogel joined the channel
GitHub117 joined the channel
NOTICE: [h] tilgovi deleted clean-up-cli at 1f07b03: https://github.com/hypothesis/h/commit/1f07b03
GitHub117 has left the channel
GitHub61 joined the channel
NOTICE: [h] tilgovi pushed 1 new commit to master: https://github.com/hypothesis/h/commit/8d9b9381d5951d994da99022dfe3b3966a3c240c