Skip to content

Dev tooling, dependencies, linting. - #2044

Open
Br3nda wants to merge 6 commits into
mysociety:masterfrom
openaustralia:feat/perl-lint
Open

Dev tooling, dependencies, linting.#2044
Br3nda wants to merge 6 commits into
mysociety:masterfrom
openaustralia:feat/perl-lint

Conversation

@Br3nda

@Br3nda Br3nda commented Jul 19, 2026

Copy link
Copy Markdown

HELLO! I've been working for OpenAustralia on long-overdue maintenance of the Australian fork. We actually forked before git was invented, back in SVN/CVS days, but are looking into being a bit closer to the UK edition.

I'd like to share some housekeeping and "developer quality of life" improvements upstream to you.

  • linting the Perl
  • versions in mise, to assist developers having the same versions of Ruby, PHP, Python, and Perl for this code (no messing with system packages)
  • cpan file to record Perl dependencies.
  • script to install dependencies

We hope you'll accept this.

@Br3nda Br3nda changed the title Adding linting for perl Dev tooling, dependencies, linting. Jul 19, 2026

@dracos dracos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello! Thanks for this :) Looks good.

Could install-deps be instead called bootstrap to match our use in other repos and https://github.com/github/scripts-to-rule-them-all?

I think perlcritic looks for pl/pm files automatically, so could the lines that run that, run perlcritic scripts search instead? Otherwise it feels like it's going to produce a lot more output than we'd want :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants