Skip Menu |

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

Report information
The Basics
Id: 59880
Status: resolved
Priority: 0/
Queue: Perl-Critic-Bangs
People
Owner: Nobody in particular
Requestors: perl [...] galumph.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Fri Jul 30 11:16:53 2010 perl [...] galumph.com - Ticket created
CC:
Subject: Missing dependency upon Perl::Critic::Utils::PPIRegexp
Date: Fri, 30 Jul 2010 15:16:44 +0000 (GMT)
To: bug-perl-critic-bangs [...] rt.cpan.org
From: perl [...] galumph.com
Perl::Critic::Utils::PPIRegexp is no longer part of the Perl-Critic distribution; it's been moved to Perl-Critic-Deprecated. At minimum, you need to declare a dependency upon this module. (Really, you should declare dependencies upon all modules you use, including currently core ones.) Better would be to replace whatever you're using from it with methods that are now native to PPI itself or to use PPIx::Regexp.
  Fri Sep 17 15:52:48 2010 PETDANCE [...] cpan.org - Correspondence added
Fixed.
  Fri Sep 17 15:52:48 2010 The RT System itself - Status changed from 'new' to 'open'
  Fri Sep 17 15:52:49 2010 PETDANCE [...] cpan.org - Status changed from 'open' to 'resolved'
  Fri Sep 17 19:02:48 2010 perl [...] galumph.com - Correspondence added
Subject: Re: [rt.cpan.org #59880] Missing dependency upon Perl::Critic::Utils::PPIRegexp
Date: Fri, 17 Sep 2010 18:02:37 -0500
To: bug-Perl-Critic-Bangs [...] rt.cpan.org
From: Elliot Shank <perl [...] galumph.com>
On 9/17/10 2:52 PM, Andy Lester via RT wrote: Show quoted text
> Fixed.
You have got to be kidding. A dependency upon Perl::Critic::Deprecated? An empty module? Are you nuts? Maybe I should create a Perl-Critic-Utils-PPIRegexp distribution. Or Perl-Critic-Andy-Declare-Proper-Dependencies.
  Fri Sep 17 19:02:49 2010 The RT System itself - Status changed from 'resolved' to 'open'
  Fri Sep 17 22:12:12 2010 PETDANCE [...] cpan.org - Status changed from 'open' to 'resolved'