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

BUG: provide filename when reading a non-archived file from mail #2021

Merged
1 commit merged into from
Jul 28, 2021

Conversation

kapalex
Copy link
Contributor

@kapalex kapalex commented Jul 25, 2021

Please read the notes below and replace them with the description of you pull request

Make sure you follow the instructions in the Developer Guide - it describes how to run the test suite and which coding rules to follow.

Commits

Please review your commits before creating a pull request. We try to keep our commit history clean.
If you had to fix something in your code and added another commit on top of your existing work, please squash your commits.
This makes it a lot easier to find out why things work the way they do later on.

But please also keep in mind that one commit should only contain changes that belong together.

The cardinal rule for creating good commits is to ensure there is only one "logical change" per commit

Thus, if your changes affect multiple files and also contains tests and documentation (which it should), it might make sense to have the tests and the documentation in a separate commit.

Commit Messages

Please explain your changes also in the git commit message.
The commit message should contain a subject that gives an overview of the changes in the commit and is limited to 72 characters.
It should start with a capital letter and it should not end with a period.
Below the subject should be, separated by an empty line, the body of the commit message. The body should explain what the commit changes and why it changes thing the way it does.
Explain your modification and also explain why you didn't chose a different approach.
See also How To Write a Git Commit Message.

Description

If you are following the commit message guidelines above, all the relevant information should already be part of the commit message.
If there is anything else you want to add, feel free to do this here.

@codecov-commenter
Copy link

codecov-commenter commented Jul 25, 2021

Codecov Report

Merging #2021 (5994d26) into develop (b6f807e) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##           develop    #2021   +/-   ##
========================================
  Coverage    75.99%   76.00%           
========================================
  Files          423      423           
  Lines        22778    22778           
  Branches      3022     3022           
========================================
+ Hits         17311    17312    +1     
  Misses        4758     4758           
+ Partials       709      708    -1     
Impacted Files Coverage Δ
...elmq/bots/collectors/mail/collector_mail_attach.py 76.47% <0.00%> (ø)
intelmq/bots/experts/ripe/expert.py 77.55% <0.00%> (+1.02%) ⬆️

@ghost ghost added this to the 3.0.1 milestone Jul 27, 2021
@ghost ghost self-assigned this Jul 27, 2021
@ghost ghost added component: bots feature request Indicates new feature requests labels Jul 27, 2021
@ghost ghost merged commit aee2918 into certtools:develop Jul 28, 2021
ghost pushed a commit that referenced this pull request Jul 28, 2021
@ghost
Copy link

ghost commented Jul 28, 2021

Thanks! Also cherry-picked for maintenance.
I added tests for this case in 8a8107e

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: bots feature request Indicates new feature requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants