Skip to content

Warning on remote errors in dependency check#174

Merged
PrajwalKrishna merged 1 commit into
mainfrom
patch-1
Aug 2, 2023
Merged

Warning on remote errors in dependency check#174
PrajwalKrishna merged 1 commit into
mainfrom
patch-1

Conversation

@PrajwalKrishna
Copy link
Copy Markdown
Contributor

We don't want PR to get blocked when dependency check is unable to download

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.

We don't want PR to get blocked when dependency check is unable to download
@PrajwalKrishna PrajwalKrishna requested a review from a team as a code owner August 2, 2023 15:24
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 2, 2023

Test Results

115 tests  ±0   115 ✔️ ±0   1m 15s ⏱️ -1s
  29 suites ±0       0 💤 ±0 
  29 files   ±0       0 ±0 

Results for commit 4363ea3. ± Comparison against base commit 6d7ec63.

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 2, 2023

Codecov Report

Merging #174 (4363ea3) into main (6d7ec63) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #174   +/-   ##
=========================================
  Coverage     79.92%   79.92%           
  Complexity      476      476           
=========================================
  Files            54       54           
  Lines          2381     2381           
  Branches        103      103           
=========================================
  Hits           1903     1903           
  Misses          420      420           
  Partials         58       58           
Flag Coverage Δ
integration 79.92% <ø> (ø)
unit 78.30% <ø> (ø)

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

@PrajwalKrishna PrajwalKrishna merged commit 27095cc into main Aug 2, 2023
@PrajwalKrishna PrajwalKrishna deleted the patch-1 branch August 2, 2023 16:12
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