perltidy now required!

[mark]

Hi all,

I've gone ahead and run perltidy on our codebase and added a test that will run and tell you if things are tidy or not. This is now required for all PRs and code we're merging.

To set up and run it locally, first make sure your modules are up to date:

bin/checkconfig.pl

This will probably ask you to install some modules, do so! Then you can run tidyall and it will automatically reformat all of your files as necessary:

perl -I$LJHOME/extlib/lib/perl5 $LJHOME/extlib/bin/tidyall -a

The first time you run this will take a while. It caches the results though so further runs will be much, much faster.

That should be it -- let me know if this works for you or if you have any issues!

Comments

[casimirian]

Thank you!

[kareila]

For those who aren't following along with changelog, it was subsequently discovered that version 20190601 significantly changed its output, so that's the minimum required version at this point.

[kareila]

Relatedly, I discovered that checkconfig.pl --install wouldn't install the newer version of perltidy if an older one already existed, so I had to do this manually: cpanm -n -L $LJHOME/extlib/ Perl::Tidy

[(Anonymous)]

Hi!
There are NO 'extlib' folder! Where can I get it? Thanx.