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

Ubiquiti shadowserver parser #1383

Merged
14 commits merged into from
Mar 21, 2019
Merged

Conversation

tomas321
Copy link
Contributor

@tomas321 tomas321 commented Mar 5, 2019

No description provided.

@codecov-io
Copy link

codecov-io commented Mar 5, 2019

Codecov Report

Merging #1383 into develop will increase coverage by 0.03%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #1383      +/-   ##
===========================================
+ Coverage    74.67%   74.71%   +0.03%     
===========================================
  Files          305      306       +1     
  Lines        13879    13899      +20     
  Branches      1868     1869       +1     
===========================================
+ Hits         10364    10384      +20     
  Misses        3098     3098              
  Partials       417      417
Impacted Files Coverage Δ
intelmq/bots/parsers/shadowserver/config.py 89.16% <100%> (+0.09%) ⬆️
...s/parsers/shadowserver/test_accessible_ubiquiti.py 100% <100%> (ø)

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Looks good for me. Any further comments @th-certbund ?

@ghost ghost self-assigned this Mar 19, 2019
@ghost ghost added this to the 1.1.2 milestone Mar 19, 2019
@ghost ghost added feature Indicates new feature requests or new features component: bots labels Mar 19, 2019
@th-certbund
Copy link
Contributor

Looks good for me. Any further comments @th-certbund ?
No, I'm fine with it.

@ghost ghost merged commit 4a0cc52 into certtools:develop Mar 21, 2019
@ghost
Copy link

ghost commented Mar 21, 2019

Thanks Tomas. Merged into maintenance and develop

Follow-up commit: 43f8fc7

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 Indicates new feature requests or new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants