search

Perl-Critic / PPI

53 stars 44 forks source link

Continuous Integration and Code Coverage #164

Closed guillaumeaubert closed 2 years ago

guillaumeaubert commented 9 years ago

Given the importance of PPI, it would be nice to have the tests run automatically on a wide array of Perl versions. A search through the open issues didn't show relevant discussions, but I apologize if this has already been discussed in the past and rejected.

Travis-CI seems to be a popular option for continuous integration in the Perl community, so I'm offering a patch to integrate with it. You can see here how it would look: https://travis-ci.org/guillaumeaubert/PPI. The only additional work required would be for @adamkennedy to enable Travis-CI for his account (since PPI's repository is tied to his account).

I've also included support for Coveralls.io, another popular option for code coverage analysis (using Devel::Cover behind the scenes). You can see it in action https://coveralls.io/r/guillaumeaubert/PPI, and @adamkennedy would need to activate it for his account as well.

If this is merged, it will be trivial to add badges to the readme file, to indicate the build status and coverage % at a glance.

As a side note - I had to manually add Test::ClassAPI to the prerequisites, so the logic xt/api.t should probably be fixed.

Thank you for considering this pull request!

wchristian commented 7 years ago

@adamkennedy The code for this is in master, but the Travis part cannot work without your action. Can you please go to this website, connect with github, and enable functionality for this repository?

https://travis-ci.org/profile/

oalders commented 2 years ago

Thanks very much for your work @guillaumeaubert! I'm sorry that it never got merged. CI is now running on GitHub actions and Travis has not been great for a while now, so I'm going to close this.