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: filter out junk mail images #742

Merged
merged 2 commits into from
Nov 1, 2022
Merged

fix: filter out junk mail images #742

merged 2 commits into from
Nov 1, 2022

Conversation

firstof9
Copy link
Collaborator

@firstof9 firstof9 commented Nov 1, 2022

Proposed change

Filter out junk mail images.

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (which adds functionality)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests
  • Documentation update
  • Adds a new shipper
  • Update existing shipper

Additional information

  • This PR is related to issue: n/a

@firstof9 firstof9 added the bugfix Pull request to fix a bug label Nov 1, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #742 (0a91fdc) into dev (9f1db97) will decrease coverage by 0.14%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #742      +/-   ##
==========================================
- Coverage   97.43%   97.29%   -0.15%     
==========================================
  Files           6        6              
  Lines        1288     1295       +7     
==========================================
+ Hits         1255     1260       +5     
- Misses         33       35       +2     
Impacted Files Coverage Δ
custom_components/mail_and_packages/helpers.py 96.62% <100.00%> (-0.23%) ⬇️

@firstof9 firstof9 merged commit 845c0eb into moralmunky:dev Nov 1, 2022
@firstof9 firstof9 deleted the filter-junk-mail branch November 1, 2022 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Pull request to fix a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants