Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Perl-Critic CPAN distribution.

Report information
The Basics
Id: 25449
Status: resolved
Priority: 0/
Queue: Perl-Critic

People
Owner: Nobody in particular
Requestors: yuki-o [...] is.naist.jp
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Proposal of $VERSION declaration
Date: Thu, 15 Mar 2007 15:08:23 +0900
To: bug-Perl-Critic [...] rt.cpan.org
From: "OHMAE Yuki" <yuki-o [...] is.naist.jp>
Greetings, Why don't you permit this idiom? use version; use Readonly; Readonly our $VERSION => qv('0.1.1'); Perl::Critic::Policy::Modules::RequireVersionVar does not recognize this, but I think declaring all-upper-case variable as a constant is a best practice. Under Perl-Critic-1.01, Perl 5.8.8, Windows XP. Regards, -- 大前 勇輝 (OHMAE Yuki) Systems Biology Lab, Nara Institute of Science and Technology
This has been fixed in Perl-Critic-1.04 which has just been uploaded to CPAN. Thanks for reporting this! -Jeff