Add shebang line to test scripts #27

Merged
merged 1 commit into from Apr 10, 2017

Conversation

Projects
None yet
4 participants
@paultcochrane
Contributor

paultcochrane commented Nov 11, 2016

Perl::Critic (at severity level 4) warns that these files don't contain
a package declaration. It is considered by Perl::Critic that any file
that isn't a module is a program, and hence needs a shebang line
pointing to the perl interpreter to denote this. This change silences
this warning.

Add shebang line to test scripts
Perl::Critic (at severity level 4) warns that these files don't contain
a `package` declaration.  It is considered by Perl::Critic that any file
that isn't a module is a program, and hence needs a shebang line
pointing to the perl interpreter to denote this.  This change silences
this warning.
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Nov 11, 2016

Coverage Status

Coverage remained the same at 99.492% when pulling ae0e29e on paultcochrane:pr/add-shebang-to-tests into 7d29212 on fayland:master.

coveralls commented Nov 11, 2016

Coverage Status

Coverage remained the same at 99.492% when pulling ae0e29e on paultcochrane:pr/add-shebang-to-tests into 7d29212 on fayland:master.

@codecov-io

This comment has been minimized.

Show comment
Hide comment
@codecov-io

codecov-io Nov 11, 2016

Current coverage is 93.40% (diff: 100%)

Merging #27 into master will not change coverage

@@             master        #27   diff @@
==========================================
  Files            14         14          
  Lines           197        197          
  Methods           0          0          
  Messages          0          0          
  Branches         26         26          
==========================================
  Hits            184        184          
  Misses            1          1          
  Partials         12         12          

Powered by Codecov. Last update 7d29212...ae0e29e

codecov-io commented Nov 11, 2016

Current coverage is 93.40% (diff: 100%)

Merging #27 into master will not change coverage

@@             master        #27   diff @@
==========================================
  Files            14         14          
  Lines           197        197          
  Methods           0          0          
  Messages          0          0          
  Branches         26         26          
==========================================
  Hits            184        184          
  Misses            1          1          
  Partials         12         12          

Powered by Codecov. Last update 7d29212...ae0e29e

@fayland fayland merged commit 240d7fa into fayland:master Apr 10, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment