Skip to content

Conversation

apaparazzi0329
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Mar 18, 2022

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 90.29622% with 95 lines in your changes missing coverage. Please review.

Project coverage is 91.98%. Comparing base (21399b7) to head (773effe).
Report is 77 commits behind head on master.

Files with missing lines Patch % Lines
ibm_watson/discovery_v1.py 84.42% 45 Missing ⚠️
ibm_watson/assistant_v1.py 92.28% 30 Missing ⚠️
ibm_watson/assistant_v2.py 91.98% 19 Missing ⚠️
ibm_watson/speech_to_text_v1.py 97.61% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #816      +/-   ##
==========================================
- Coverage   92.00%   91.98%   -0.02%     
==========================================
  Files          25       19       -6     
  Lines       31988    26076    -5912     
==========================================
- Hits        29429    23986    -5443     
+ Misses       2559     2090     -469     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lgtm-com
Copy link

lgtm-com bot commented Mar 18, 2022

This pull request fixes 1 alert when merging d0262f3 into 21399b7 - view on LGTM.com

fixed alerts:

  • 1 for Unused import

Copy link

@kevinkowa kevinkowa left a comment

Choose a reason for hiding this comment

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

LGTM after detect-secrets

@lgtm-com
Copy link

lgtm-com bot commented Mar 21, 2022

This pull request fixes 1 alert when merging 5eb3eab into 21399b7 - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Mar 21, 2022

This pull request fixes 1 alert when merging b9d4b28 into 21399b7 - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Mar 21, 2022

This pull request fixes 1 alert when merging 773effe into 21399b7 - view on LGTM.com

fixed alerts:

  • 1 for Unused import

Copy link

@kevinkowa kevinkowa left a comment

Choose a reason for hiding this comment

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

lgtm

@apaparazzi0329 apaparazzi0329 merged commit 97de097 into master Mar 21, 2022
@watson-github-bot
Copy link
Collaborator

🎉 This PR is included in version 6.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

5 participants