> On 5 Jun 2018, at 16:31, Andrew Dunstan <andrew.duns...@2ndquadrant.com> > wrote:
> The patch contains a simple script to run the checks. The code that finds > perl files is put in a function in a single file that is sourced by the three > locations that need it. +1 on centralizing the find-files function. > The directory pgperlcritic is renamed to perlcheck, as it not contains the > new script as well as pgperlcritic. This comment should say “perlcheck/..” and not “pgperlcritic/.." I assume: --- /dev/null +++ b/src/tools/perlcheck/pgperlcritic @@ -0,0 +1,20 @@ +#!/bin/sh + +# src/tools/pgperlcritic/pgperlcritic cheers ./daniel