hypertrace / hypertrace-ui

UI for Hypertrace
Other
25 stars 12 forks source link

feat: add disabled mode to splitter #2552

Closed jake-bassett closed 10 months ago

jake-bassett commented 10 months 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 10 months ago

Test Results

       4 files  ±0     316 suites  ±0   31m 41s :stopwatch: - 1m 27s 1 135 tests ±0  1 133 :heavy_check_mark:  - 2  0 :zzz: ±0  2 :x: +2  1 145 runs  ±0  1 143 :heavy_check_mark:  - 2  0 :zzz: ±0  2 :x: +2 

For more details on these failures, see this check.

Results for commit fd4da580. ± Comparison against base commit 56f3457a.

codecov[bot] commented 10 months ago

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (56f3457) 82.20% compared to head (fd4da58) 74.48%.

:exclamation: Current head fd4da58 differs from pull request most recent head e3ad1e5. Consider uploading reports for the commit e3ad1e5 to get more accurate results

Files Patch % Lines
...ects/components/src/splitter/splitter.component.ts 0.00% 4 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2552 +/- ## ========================================== - Coverage 82.20% 74.48% -7.73% ========================================== Files 927 927 Lines 20860 20863 +3 Branches 3310 3311 +1 ========================================== - Hits 17149 15539 -1610 - Misses 3574 5109 +1535 - Partials 137 215 +78 ```

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