search

garu / App-cpanminus-reporter

stand-alone CPAN Testers client (for cpanminus and friends)
13 stars 13 forks source link

Publish multiple reports #21

Closed Mekk closed 1 year ago

Mekk commented 9 years ago

I offer two new options (sorry for bundling them together but I added --max-age after trying to use --all):

--all causes cpanm-reporter to consider not only last build.log, but also other recent build.log files present in ~/cpanm/work. I wrote it as I routinely cpanm OnePackage cpanm SecondOne cpanm-reporter # Ops, only Second reported

--max-age=60 specify max allowed age of build.log, in minutes. This is safer than full-blown force, especially in case --all is being used.

There are also some tweaks in doc and messages to accomodate those changes, and minor refactoring to make processing multiple build.log's easier.

coveralls commented 9 years ago

Coverage Status

Coverage decreased (-2.23%) to 88.11% when pulling 2aae227696b139ee2cd72c4d775d33d34e0a1c01 on Mekk:publish_multiple_reports into 77f65fc554a75267836a14836f35152542e0a170 on garu:master.

garu commented 1 year ago

Thank you, @Mekk! Sorry for the huge delay, took me a while to get back to this module.

Your patch is merged and I really like the new feature. I'll just tweak it a bit more before the actual release because I don't want to add non-core modules like Path::Tiny to this distribution.

Thanks again!