New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Various minor fixes #18

Merged
merged 7 commits into from Jun 25, 2017

Conversation

Projects
None yet
3 participants
@lancew
Contributor

lancew commented Jun 25, 2017

No description provided.

lancew added some commits Jun 25, 2017

Add use strict and use warnings in 00-load.t
Minor fix picked up by kritika.io.
Remove trailing whitespace from PGObject.pm
My vim setup highlighted this very minor tidy up.
Remove trailing whitespace in files
Makefile.pl and Registry.pm also had some trailing white spaces so
removed them in this commit.
Modify handling of @_ in subs to be consistent
Changed all handling in PGObject of @_ to be done same way:
    my ($self, %args) = @_;
Apply Perltidy to /lib
Ran Perltidy over PGObject.pm and Registry.pm
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jun 25, 2017

Coverage Status

Coverage increased (+0.06%) to 97.619% when pulling 08b634d on lancew:master into cd2cda0 on ledgersmb:master.

coveralls commented Jun 25, 2017

Coverage Status

Coverage increased (+0.06%) to 97.619% when pulling 08b634d on lancew:master into cd2cda0 on ledgersmb:master.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jun 25, 2017

Coverage Status

Coverage increased (+0.06%) to 97.619% when pulling 08b634d on lancew:master into cd2cda0 on ledgersmb:master.

coveralls commented Jun 25, 2017

Coverage Status

Coverage increased (+0.06%) to 97.619% when pulling 08b634d on lancew:master into cd2cda0 on ledgersmb:master.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jun 25, 2017

Coverage Status

Coverage increased (+0.06%) to 97.619% when pulling 08b634d on lancew:master into cd2cda0 on ledgersmb:master.

coveralls commented Jun 25, 2017

Coverage Status

Coverage increased (+0.06%) to 97.619% when pulling 08b634d on lancew:master into cd2cda0 on ledgersmb:master.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jun 25, 2017

Coverage Status

Coverage increased (+0.06%) to 97.619% when pulling 08b634d on lancew:master into cd2cda0 on ledgersmb:master.

coveralls commented Jun 25, 2017

Coverage Status

Coverage increased (+0.06%) to 97.619% when pulling 08b634d on lancew:master into cd2cda0 on ledgersmb:master.

@einhverfr einhverfr merged commit 4296065 into ledgersmb:master Jun 25, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.06%) to 97.619%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment