[16:07:23] @paladox why https://github.com/RhinosF1/puppet/commit/cf91919a6bd8a727fd3c9421e5ccada8b435f299 [16:09:12] because they are huge and we already store sql dumps, no point doing those as well. (Cost a lot). I have https://github.com/miraheze/puppet/pull/3357 but it's not ready yet. [16:40:26] @paladox -1'd [16:41:57] It looks like pca_password does it [16:42:11] I would change it to using the already existing arg parse setup [16:55:53] It’s already like that before that change [18:17:37] @paladox yes but why not fix now [18:17:41] it's a 2 line change [18:32:43] (also means we'll be able to run CI on the script)