Skip to content

Conversation

sarthak77
Copy link
Member

No description provided.

@github-actions

This comment has been minimized.

@codecov
Copy link

codecov bot commented Oct 6, 2022

Codecov Report

Merging #37 (9d463c6) into main (1f5edb2) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main      #37   +/-   ##
=========================================
  Coverage     72.43%   72.43%           
  Complexity      129      129           
=========================================
  Files            19       19           
  Lines           370      370           
  Branches         21       21           
=========================================
  Hits            268      268           
  Misses           81       81           
  Partials         21       21           
Flag Coverage Δ
unit 72.43% <ø> (ø)

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

skjindal93
skjindal93 previously approved these changes Oct 6, 2022
Comment on lines 15 to 17
constraints {
implementation("com.fasterxml.jackson.core:jackson-databind:2.13.4")
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add the constraints section below the implementations? Please also add the reason for the constraint. You can add a link for the CVE like

constraints {
  implementation("com.fasterxml.jackson.core:jackson-databind:2.13.4") {
    because("https://nvd.nist.gov/vuln/detail/CVE-2022-42004")
  }
}

@github-actions

This comment has been minimized.

@sarthak77 sarthak77 merged commit 1e3ae39 into main Oct 6, 2022
@sarthak77 sarthak77 deleted the adding/dependency/constraint branch October 6, 2022 06:50
@github-actions
Copy link

github-actions bot commented Oct 6, 2022

Unit Test Results

12 files  ±0  12 suites  ±0   16s ⏱️ -3s
61 tests ±0  61 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 1e3ae39. ± Comparison against base commit 1f5edb2.

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.

3 participants