Adding an EPIC config file

Dealing with more Eclipse files

Added a build.xml script to all distributions.  Tools like Eclipse and Intelli-J
have good interfaces for creating and running Ant scripts.  These files
just serve as wrappers around Build.PL, so you can take advantage of those
interfaces when working with Perl dists.

Taking premptive steps: adding my forthcoming build.xml to MANIFEST.SKIP

Fix PBP page numbers in ProhibitMixedCaseVars and
ProhibitMixedCaseVars.

Forgot to set svn:keywords on
RegularExpressions::RequireDotMatchAnything.

Make tests comply with
RegularExpressions::RequireDotMatchAnything.

Make rest of code comply with
RegularExpressions::RequireDotMatchAnything.

Make Policies comply with
RegularExpressions::RequireDotMatchAnything.

Implement RegularExpressions::RequireDotMatchAnything.

Create an option for CodeLayout::ProhibitQuotedWordLists
that makes it complain about word lists containing non-word
characters again.

Add ".project" to all of the MANIFEST.SKIP files.

Color for high-severity violations is now magenta,
because this looks better on typical terminals with
a white background.  RT #38511.

Adding username for anonymous checkout from our repository.
Thanks to Schwern for suggesting.  #RT 38628.

I started using Eclipse for all my Perl work.  So I am checking in the Eclipse project files for each distro.  I am not really sure if these should be in the repository or not

Undoing r2675.  Now that the "-color" is no longer being set in _parse_command_line(), we no longer need to worry about double dashes in the keys of the argument hash for P::C->new().  Default values for the command-line options should only be defined in the OptionsProcessor.

Added failing test case that came to me in the mail today.

Change perlcritic to not unconditionally add a "-" to the
front of options to be passed to Perl::Critic::new() in
response to RT #38458.

Make the changes to
RegularExpressions::RequireExtendedFormatting more clear in
Changes.

Clean up Schwern's
RegularExpressions::RequireExtendedFormatting patch. Renamed
the min_regex_length option to
minimum_regex_length_to_complain_about. The way the
allow_with_whitespace was coded, if the regex contained any
whitespace at all, it was exempt. Instead, I made regexes
that only contain word characters and whitespace exempt by
default, and added a strict option.  I'm not so sure about
this one.

Apply raw patch from Schwern that adds options to
RegularExpressions::RequireExtendedFormatting.

Fix missing word in the ProhibitNoWarnings POD.

Update Changes with the new allow_with_category_restriction
option for TestingAndDebugging::ProhibitNoWarnings.

Clean up both existing problems and problems with Schwern's
patch in TestingAndDebugging::ProhibitNoWarnings.

Apply raw patch from Schwern for
TestingAndDebugging::ProhibitNoWarnings.  Needs some rework.
The special value in the allow option?  Eww.  Not too hard
to fix, though.

Clean up self compliance problems introduced by Schwern's
pager patch.  Still need to actually review the patch.  :]

Apply the unmodified patch from Schwern (modulo dealing with
RJBS' change to support PAR).  I have no idea whether tests
or anything works at this point.  I just considered it
worthwhile to capture the change as it is before modifying
it further.

Comment in P::C::Document about underscores in perl version
dependencies, i.e. "use 5.008_005;".

Add the xt directory to no_index in Build.PL.

Update Changes to reflect the fix for RT #38380 that RJBS
did.

bugfix for running under PAR, RT #38380

in 20_policies.t, specify a more useful format for
violations that are shown when there's a test failure.

Inspired by failures in
Variables/RequireLocalizedPunctuationVars.run occurring when
running on a 5.10.0 + patches perl.

Another example of a problem in
ProhibitEscapedMetacharacters with character classes.

Failing test for
RegularExpressions::ProhibitEscapedMetacharacters.

Forgot to update Changes

Ternaries now satisfy RequireCheckingReturnValueOfEval:
  eval { require Foo; 1; } ? ok() : nok();

Describe policy for enforcing correct spelling of variable names.

This all got started here: http://rt.cpan.org/Ticket/Display.html?id=37899

Fixed spelling error. RT #37899

CodeLayout::ProhibitQuotedWordLists also now applies to the
import arguments of a C<use> statement.  RT #24467.

I also cleaned up some of the tests for this Policy.

Disabled RequireConstantOnLeftSideOfEquality until I decide
which distro to put that policy in.

ProhibitQuotedWordLists does not apply to lists
that contain non-words. RT #37886.

Add RT # to ProhibitPostfixControls TODO.

TODO about ControlStructures::ProhibitPostfixControls
changing its explanation when "do {} while" is found.

Diddling svn:ignore.

Added my first new policy in a very long time.  So I might be a bit
rusty here...

This is a partial implementation of the RequireConstantBeforeOperator
policy mentioned in the TODO list.  For now, it only applies to the
'==' and 'eq' operators.  But if you guys think it should be extended
to all other comparison operators, then I can do that.

20_policies.t didn't have a "1;" at the end, so it failed to
load in 20_policies.t_without_optional_dependencies.t.

Bump P::C version to 1.090.

ProhibitSingleCharAlternation test was failing when
Regexp::Parser wasn't installed, incorrectly.  In addition
to fixing the test, generate a
"_without_optional_dependencies" wrapper around
t/20_policies.t.

Bump P::C version to 1.089.

Knock out RequirePodSections in 20_policy_podspelling.t.
I don't understand why but I can't knock it out in
42_perlcriticrc-tests.  However, I want to get a release
out, so I'll get back to it later.

Fix ProhibitPostfixControls in tests.

Deal with RequireInterpolationOfMetachars in tests.

Fix ProhibitEscapedMetacharacters in tests.

Fix ProhibitEscapedCharacters in
20_policy_requireconsistentnewlines.t.

Fix noisy strings in tests.

Fix ProhibitInterpolationOfLiterals in tests.

Fix RequireBracesForMultiline in 01_config_bad_perlcriticrc.t.

Fix ProhibitEnumeratedClasses in 94_includes.t.

Fix PodSpelling issues in tests.  In t/05_utils_pod.t, I had
to "## no critic" it because I couldn't get stopwords to
work for "cperl".

Add rcs_keywords option to RequireInterpolationOfMetachars.

Knock out ProhibitMagicNumbers for the tests right now.

POD formatting cleanups in DEVELOPER.pod and minor doc edit
in P::C::Violation.

RT #34713.  Tweaked Subroutines::ProtectPrivateVars
so that "shift->_private_sub()" is allowed.

Add exec to the list of things accepted by
RequireFinalReturn.

Additional stopwords for the code examples.

Subroutines::ProhibitBuiltinHomonyms now covers Perl keywords
as well.  Inspired by RT #37632

Change name of option in ProhibitInterpolationOfLiterals to
"allow_if_string_contains_single_quote".

Fix PodSpelling problems caused by the doubling up of L< >
contents to get around the "manpage" problem.

Add allow_double_quote_if_string_contains_single_quote
option to ProhibitInterpolationOfLiterals.  Patch submitted
by Ed Avis (plus further edits).  RT #36125.

Remove documentation about the -config option to P::C::new()
so that people won't start new code that depends upon it.

Fix RequireLineBoundaryMatching in 99_pod_coverage.t.

Fix ProhibitPunctuationVars in tests.

Fix ProhibitLocalVars in 20_policy_podspelling.t.  Someone
was attempting to localize a lexical.

Fix ProhibitEmptyQuotes in tests.

Fix ProhibitDoubleSigils in tests.

Fix RequireRcsKeywords in tests.

Fix RequireCheckedClose in 05_utils.t.

Fix RequireFinalReturn in 05_utils.t.

The for loop TODO test for RequireFinalReturn wasn't quite
valid: You've got to name the loop variable.

Add TODO test for Subroutines::RequireFinalReturn having to
do with PPI's mis-parsing of "for qw<blah> {}".

is_script() now tries to check if the filename ends in ".PL".
This prevents it from misclassifying Makefile.PL and Build.PL
and other ".PL" scripts as modules.  But if the document is
coming from a pipe (rather than a file) then the filename
won't be available and is_script() will still misclassify
the file.  Another approach may be to look for the absence
of a "package" statement, but I'm not sure that is very
reliable.  I suspect that many newbie developers and those
with very old legacy systems may have lots of libraries that
don't have any package statements.

Make -s a synonym for --single-policy.

Fix RequireCheckedClose in tests.

Knock out RegularExpressions::ProhibitComplexRegexes and
ErrorHandling::RequireCheckingReturnValueOfEval for tests.

Finish fixing RequireExtendedFormatting problems.

More RequireExtendedFormatting fixes in tests.

Fix RequireExtendedFormatting in
01_config_bad_perlcriticrc.t.

Fix RequireExtendedFormatting in 01_config.t.

Fix the last of the RequireTestLabels problems.

Fixed typo in the code for setting the criticism-fatal option.  Damn I hate
all this shennanigans with hyphens and underscores.

Update class diagram to mark more classes as public.

Fix RequireTestLabels in 09_theme.t.

Add --list output formatting to TODO.pod.

Fix RequireTestLabels problems in 02_policy.t.

Finish RequireTestLabels fixes in 10_userprofile.t.

Fix ProhibitAmpersandSigils in the "sort &foo(...)" case.

Revert ProhibitParensWithBuiltins test change. I was looking
at the problem wrong.

Failing "(caller(0))[3]" test for
ProhibitParensWithBuiltins.

Failing test for ProhibitAmpersandSigils with sort.

Convert L<Some::Module> escapes to
L<Some::Module|Some::Module> in response to RT #37485.

Fix false positive in ProhibitSingleCharAlternation