Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: fix deepsource warnings #1174

Merged
merged 4 commits into from
Aug 20, 2023
Merged

fix: fix deepsource warnings #1174

merged 4 commits into from
Aug 20, 2023

Conversation

raphael0202
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Aug 17, 2023

Codecov Report

Merging #1174 (a6e7f86) into main (7ab981d) will decrease coverage by 0.02%.
Report is 2 commits behind head on main.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main    #1174      +/-   ##
==========================================
- Coverage   56.14%   56.12%   -0.02%     
==========================================
  Files          79       79              
  Lines        8150     8142       -8     
==========================================
- Hits         4576     4570       -6     
+ Misses       3574     3572       -2     
Files Changed Coverage Δ
robotoff/cli/logos.py 0.00% <0.00%> (ø)
robotoff/cli/main.py 14.64% <0.00%> (-1.30%) ⬇️
robotoff/metrics.py 22.72% <ø> (+0.31%) ⬆️
...y/neural/keras_category_classifier_3_0/__init__.py 73.75% <0.00%> (ø)
...tion/object_detection/utils/visualization_utils.py 19.20% <0.00%> (ø)
robotoff/products.py 41.75% <0.00%> (ø)
robotoff/taxonomy.py 72.72% <0.00%> (ø)
robotoff/app/api.py 47.76% <100.00%> (ø)
robotoff/insights/extraction.py 51.21% <100.00%> (ø)
robotoff/prediction/ocr/product_weight.py 47.82% <100.00%> (ø)
... and 2 more

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

Copy link
Member

@teolemon teolemon left a comment

Choose a reason for hiding this comment

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

Recent Changes didn't work anymore ?

@raphael0202
Copy link
Collaborator Author

Recent Changes ?

@raphael0202 raphael0202 merged commit 1aa13ad into main Aug 20, 2023
7 of 9 checks passed
@raphael0202 raphael0202 deleted the fix-deepsource-warnings branch August 20, 2023 17:46
@teolemon
Copy link
Member

Screenshot_20230820-204043.png

@raphael0202
Copy link
Collaborator Author

It was not used anywhere!

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

Successfully merging this pull request may close these issues.

2 participants