Skip to content

Add missing Log4J dependency due to Scio 0.13.3 bug#671

Merged
monzalo14 merged 1 commit into
spotify:masterfrom
monzalo14:patch-log4j-dep
Sep 21, 2023
Merged

Add missing Log4J dependency due to Scio 0.13.3 bug#671
monzalo14 merged 1 commit into
spotify:masterfrom
monzalo14:patch-log4j-dep

Conversation

@monzalo14

@monzalo14 monzalo14 commented Sep 21, 2023

Copy link
Copy Markdown
Member

Fixes #672

@codecov

codecov Bot commented Sep 21, 2023

Copy link
Copy Markdown

Codecov Report

Merging #671 (90bb65a) into master (70c6157) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #671   +/-   ##
=======================================
  Coverage   71.40%   71.40%           
=======================================
  Files          40       40           
  Lines        1710     1710           
  Branches      140      140           
=======================================
  Hits         1221     1221           
  Misses        489      489           
Flag Coverage Δ
ratatoolCli 3.07% <ø> (ø)
ratatoolCommon ∅ <ø> (?)
ratatoolDiffy 30.80% <ø> (ø)
ratatoolExamples 15.94% <ø> (ø)
ratatoolSampling 62.58% <ø> (ø)
ratatoolScalacheck 81.67% <ø> (ø)
ratatoolShapeless 4.41% <ø> (ø)

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

@monzalo14
monzalo14 merged commit c4cefd2 into spotify:master Sep 21, 2023
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.

Missing Log4J dependency

2 participants