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
Add GitHub Actions. #3244
Add GitHub Actions. #3244
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3244 +/- ##
==========================================
+ Coverage 84.23% 84.41% +0.18%
==========================================
Files 264 264
Lines 16700 16703 +3
Branches 3124 3125 +1
==========================================
+ Hits 14067 14100 +33
+ Misses 1636 1581 -55
- Partials 997 1022 +25
Continue to review full report at Codecov.
|
9155710
to
bfea469
Compare
We now have Image::Magick, so make sure that is handled better.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤖
PERL5LIB: local-coverage/lib/perl5 | ||
HARNESS_PERL_SWITCHES: "-MDevel::Cover=+ignore,local/lib/perl5,commonlib,perllib/Catalyst/[^A],perllib/Email,Test.pm,^t" | ||
|
||
- name: Run tests (with coverage, part 2) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why the two parts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So it finishes quicker overall. It's a bit lopsided as you can see from the runs, but was at least an easy place to split them. Now it's in its own workflow, we could consider splitting further...
Originally done in f7873a9
[skip changelog]
Diff failure is because of the changes added to deal with perlmagick being installed now and it trying to shrink the 1x83 test image which fails. OTOH is now testing e.g. the static_map stitch code which it wasn't on Travis.
Tried using the perl version docker images, but had weird test failures (slim base builds? Couldn't work it out), so instead am using GitHub's default, perlbrew to install a specific version and caching that.