hypertrace / hypertrace-ingester

Streaming jobs for Hypertrace
Other
13 stars 16 forks source link

chore | Validate regex before persisting #393

Closed sanket-mundra closed 1 year ago

sanket-mundra commented 1 year ago

Description

Please include a summary of the change, motivation and context.

Testing

Please describe the tests that you ran to verify your changes. Please summarize what did you test and what needs to be tested e.g. deployed and tested helm chart locally.

Checklist:

Documentation

Make sure that you have documented corresponding changes in this repository or hypertrace docs repo if required.

github-actions[bot] commented 1 year ago

Unit Test Results

  77 files  ±0    77 suites  ±0   1m 32s :stopwatch: -5s 416 tests ±0  416 :heavy_check_mark: ±0  0 :zzz: ±0  0 :x: ±0 

Results for commit 2f48ea6a. ± Comparison against base commit b6b4d952.

codecov[bot] commented 1 year ago

Codecov Report

Merging #393 (2f48ea6) into main (b6b4d95) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #393   +/-   ##
=========================================
  Coverage     80.07%   80.07%           
  Complexity     1416     1416           
=========================================
  Files           128      128           
  Lines          5546     5546           
  Branches        510      510           
=========================================
  Hits           4441     4441           
  Misses          870      870           
  Partials        235      235           
Flag Coverage Δ
unit 80.07% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

github-actions[bot] commented 1 year ago

Unit Test Results

  77 files  ±0    77 suites  ±0   1m 30s :stopwatch: -7s 416 tests ±0  416 :heavy_check_mark: ±0  0 :zzz: ±0  0 :x: ±0 

Results for commit 123a2da7. ± Comparison against base commit b6b4d952.