From: Peter Rabbitson Date: Wed, 11 Apr 2012 07:24:01 +0000 (+0200) Subject: Check more files for notabs/eol X-Git-Tag: v0.08197~40 X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=a8b66abc9cf90221ebcd9be535eda47f6b37a6f8;hp=20884f55ed03738633c1cf2a1f8d43af743b650e;p=dbsrgits%2FDBIx-Class.git Check more files for notabs/eol --- diff --git a/examples/Schema/testdb.pl b/examples/Schema/testdb.pl index c608f45..a65db0f 100644 --- a/examples/Schema/testdb.pl +++ b/examples/Schema/testdb.pl @@ -55,7 +55,6 @@ sub get_tracks_by_artist { print "\n"; } - sub get_cd_by_track { my $tracktitle = shift; print "get_cd_by_track($tracktitle):\n"; @@ -88,8 +87,6 @@ sub get_cds_by_artist { print "\n"; } - - sub get_artist_by_track { my $tracktitle = shift; print "get_artist_by_track($tracktitle):\n"; @@ -107,7 +104,6 @@ sub get_artist_by_track { print $artist->name . "\n\n"; } - sub get_artist_by_cd { my $cdtitle = shift; print "get_artist_by_cd($cdtitle):\n"; diff --git a/xt/eol.t b/xt/eol.t index ab2d9a1..4baf714 100644 --- a/xt/eol.t +++ b/xt/eol.t @@ -14,10 +14,13 @@ unless ( DBIx::Class::Optional::Dependencies->req_ok_for ('test_eol') ) { } Test::EOL::all_perl_files_ok({ trailing_whitespace => 1 }, - qw/t xt lib script/, + qw/t xt lib script examples/, DBICTest::RunMode->is_author ? ('maint') : (), ); +# Changes is not a "perl file", hence checked separately +Test::EOL::eol_unix_ok('Changes', { trailing_whitespace => 1 }); + # FIXME - Test::EOL declares 'no_plan' which conflicts with done_testing # https://github.com/schwern/test-more/issues/14 #done_testing; diff --git a/xt/notabs.t b/xt/notabs.t index 7734b75..15e218f 100644 --- a/xt/notabs.t +++ b/xt/notabs.t @@ -14,10 +14,13 @@ unless ( DBIx::Class::Optional::Dependencies->req_ok_for ('test_notabs') ) { } Test::NoTabs::all_perl_files_ok( - qw/t xt lib script/, + qw/t xt lib script examples/, DBICTest::RunMode->is_author ? ('maint') : (), ); +# Changes is not a "perl file", hence checked separately +Test::NoTabs::notabs_ok('Changes'); + # FIXME - Test::NoTabs declares 'no_plan' which conflicts with done_testing # https://github.com/schwern/test-more/issues/14 #done_testing;