cloudflare / pint

Prometheus rule linter/validator
https://cloudflare.github.io/pint/
Apache License 2.0
879 stars 53 forks source link

Update semgrep.yml #1119

Closed hrushikeshdeshpande closed 2 months ago

hrushikeshdeshpande commented 2 months ago

Updating semgrep docker image for the exising semgrep workflow.

github-actions[bot] commented 2 months ago

Benchmark diff:

goos: linux
goarch: amd64
pkg: github.com/cloudflare/pint/cmd/pint
cpu: AMD EPYC 7763 64-Core Processor                
              │   old.txt   │              new.txt               │
              │   sec/op    │   sec/op     vs base               │
FindEntries-4   39.97m ± 1%   40.07m ± 3%       ~ (p=0.971 n=10)
CheckRules-4     8.490 ± 2%    8.500 ± 2%       ~ (p=0.853 n=10)
geomean         582.6m        583.6m       +0.17%

              │   old.txt    │               new.txt               │
              │     B/op     │     B/op      vs base               │
FindEntries-4   27.41Mi ± 0%   27.41Mi ± 0%  -0.01% (p=0.035 n=10)
CheckRules-4    978.8Mi ± 0%   979.3Mi ± 0%       ~ (p=0.393 n=10)
geomean         163.8Mi        163.8Mi       +0.02%

              │   old.txt   │              new.txt               │
              │  allocs/op  │  allocs/op   vs base               │
FindEntries-4   131.9k ± 0%   131.9k ± 0%  -0.00% (p=0.048 n=10)
CheckRules-4    14.20M ± 0%   14.21M ± 0%       ~ (p=0.393 n=10)
geomean         1.368M        1.369M       +0.01%
codecov-commenter commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.12%. Comparing base (f853bca) to head (02c54c8).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1119 +/- ## ======================================= Coverage 95.12% 95.12% ======================================= Files 102 102 Lines 10195 10195 ======================================= Hits 9698 9698 Misses 346 346 Partials 151 151 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.