Skip to content

use v2 log4j-api and slf4j2impl#425

Merged
satish-mittal merged 1 commit intomainfrom
log4j_upgrade
Oct 4, 2023
Merged

use v2 log4j-api and slf4j2impl#425
satish-mittal merged 1 commit intomainfrom
log4j_upgrade

Conversation

@satish-mittal
Copy link
Copy Markdown
Contributor

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:

  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • Any dependent changes have been merged and published in downstream modules

Documentation

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

@satish-mittal satish-mittal requested a review from a team as a code owner October 4, 2023 12:05
@github-actions

This comment has been minimized.

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 4, 2023

Codecov Report

Merging #425 (1b1e247) into main (52974ed) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #425   +/-   ##
=========================================
  Coverage     79.88%   79.88%           
  Complexity     1424     1424           
=========================================
  Files           128      128           
  Lines          5587     5587           
  Branches        511      511           
=========================================
  Hits           4463     4463           
  Misses          891      891           
  Partials        233      233           
Flag Coverage Δ
unit 79.88% <ø> (ø)

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

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

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Oct 4, 2023

Unit Test Results

  78 files  ±0    78 suites  ±0   1m 24s ⏱️ -2s
418 tests ±0  418 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit da0b763. ± Comparison against base commit 52974ed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants