[Koha-devel] What's the definitive way to use Perl tidy in koha-testing-docker?

Katrin Fischer katrin.fischer.83 at web.de
Sun Dec 11 14:54:18 CET 2022


Hi David,

when using perltidy in my editor with the standard configuration, the
indentation appears to be correct. But I am not sure what you mean with
:Tidy in ktd? It looks like our coding guidelines matches the defaults
for PerlTidy: n=4 and space. Could it be that some local config file is
used instead of the default?

There is a wiki page for how to use PerlTidy that we might want to
update with any findings:
https://wiki.koha-community.org/wiki/Perltidy

Katrin


On 09.12.22 04:26, David Cook wrote:
>
> Hi all,
>
> I’ve heard a few people saying lately that they wish people would use
> Perl tidy more.
>
> So I used the “:Tidy” command in koha-testing-docker on my visual
> selection and I got the following:
>
> +                $this_item->{shelves} =
>
> + Koha::Virtualshelves->get_shelves_containing_record(
>
> +                    {
>
> +                        biblionumber => $biblionumber,
>
> +                        borrowernumber => $loggedinuser,
>
> +                    }
>
> +                  );
>
> But that goes against the Indentation coding guidelines from
> https://wiki.koha-community.org/wiki/Coding_Guidelines#Indentation
>
> So… what are we supposed to be doing these days?
>
> David Cook
>
> Senior Software Engineer
>
> Prosentient Systems
>
> Suite 7.03
>
> 6a Glen St
>
> Milsons Point NSW 2061
>
> Australia
>
> Office: 02 9212 0899
>
> Online: 02 8005 0595
>
>
> _______________________________________________
> Koha-devel mailing list
> Koha-devel at lists.koha-community.org
> https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
> website :https://www.koha-community.org/
> git :https://git.koha-community.org/
> bugs :https://bugs.koha-community.org/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.koha-community.org/pipermail/koha-devel/attachments/20221211/5344b3a3/attachment.htm>


More information about the Koha-devel mailing list