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

Next/20210525/v6 #6139

Merged
merged 16 commits into from
May 25, 2021
Merged

Next/20210525/v6 #6139

merged 16 commits into from
May 25, 2021

Conversation

victorjulien
Copy link
Member

inashivb and others added 16 commits May 21, 2021 11:01
Registering parsers in Rust requires signatures to be a certain way and
compatible with C. Change signatures of all the functions.
Get rid of the C glue code and move registration completely to Rust.
Registering parsers in Rust requires signatures to be a certain way and
compatible with C. Change signatures of all the functions.
Get rid of the C glue code and move registration completely to Rust.
instead of deprecated add_stylesheet
This commit modifies the constructors to use Self and/or
Default::default() when feasible
Including the whole directory results in .deps files ending up
in the distribution archive which shouldn't be there. Instead
we have to list all the test sources individually.
That may not be the case in midstream/async configurations
@victorjulien victorjulien requested review from jasonish, norg and a team as code owners May 25, 2021 13:30
@codecov
Copy link

codecov bot commented May 25, 2021

Codecov Report

Merging #6139 (d00b755) into master (533c6ff) will decrease coverage by 0.00%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master    #6139      +/-   ##
==========================================
- Coverage   76.87%   76.87%   -0.01%     
==========================================
  Files         613      611       -2     
  Lines      186344   186209     -135     
==========================================
- Hits       143248   143139     -109     
+ Misses      43096    43070      -26     
Flag Coverage Δ
fuzzcorpus 52.79% <75.00%> (-0.03%) ⬇️
suricata-verify 50.94% <75.00%> (-0.02%) ⬇️
unittests 63.08% <50.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
5 participants