• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

jonasbn / perl-workflow / 355
93%

Build:
DEFAULT BRANCH: master
Ran 20 May 2021 07:41PM UTC
Jobs 1
Files 36
Run time 3min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

pending completion
355

push

travis-ci

GitHub
Simplify logging by removing `is_*` logging level checks before logging (#108)

* On non-interpolated logging, eliminate `is_*` checks

For non-interpolated strings, there's really no performance benefit
in checking whether the string would be logged or not: checking it
involves a single call (if there's no logging) and _not_ checking it
also involves one call (with a single parameter being passed).
In case logging *is* set to this level, the original code involves
two function calls whereas the new code still exhibits only one.

* Add indentation around log messages in line with other function calls

* Allow string interpolation while logging during initialization

During initialization, a truckload of actions are performed. It is my
expectation (to be verified) that a few string interpolations won't
make much difference in execution speed of initialization, even if the
resulting string ends up not being logged.

Two cases where logging might cause a more elaborate routine to create
the content to be logged (one call to Data::Dumper::Dumper() and the
other to 'join()' with a list of arguments) have been wrapped in a
sub{} (closure) to prevent execution when the output would not be logged.

* Allow string interpolation while logging during workflow execution

* Close #33: Add GitHub Actions support

1524 of 1866 relevant lines covered (81.67%)

35.27 hits per line

Jobs
ID Job ID Ran Files Coverage
1 355.1 20 May 2021 07:41PM UTC 0
81.67
Travis Job 355.1
Source Files on build 355
Detailed source file information is not available for this build.
  • Back to Repo
  • Travis Build #355
  • d7755270 on github
  • Prev Build on master (#861591970)
  • Next Build on master (#1003296377)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc