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

GoogleCloudPlatform / ops-agent / 6331819346 / 1
88%
master: 88%

Build:
Build:
LAST BUILD BRANCH: martijnvans-pwsh-with-tests
DEFAULT BRANCH: master
Ran 27 Sep 2023 09:55PM UTC
Files 83
Run time 1s
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

27 Sep 2023 09:51PM UTC coverage: 88.791% (+0.001%) from 88.79%
6331819346.1

Pull #1449

github

jefferbrecht
Implement __match_ field deletion via modify

The existing lua chunk for deleting the temporary __match_ fields
relies on `match` being set, otherwise the code flows into the
`return 0, 0, 0` path which indicates the record hasn't been modified.

`return 0, 0, 0` is a nice optimization to have, and I'd rather not
remove it just for the sake of having the temp field deletion code
be lua, so this commit re-implements it as a plain `modify` filter.

Open to alternatives; the main point of this is to write a test for the
failing use case and propose a fix, which we can iterate on.
Pull Request #1449: Implement __match_ field deletion via modify

8801 of 9912 relevant lines covered (88.79%)

0.96 hits per line

Source Files on job 6331819346.1
  • Tree
  • List 83
  • Changed 1
  • Source Changed 0
  • Coverage Changed 1
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 6331819346
  • aa3c3f02 on github
  • Prev Job for on jefferbrecht-fix-match-fields (#6330525389.1)
  • Next Job for on jefferbrecht-fix-match-fields (#6339515170.1)
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