Skip to content
New issue

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

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

Already on GitHub? # to your account

lua: use lua from crates.io rather than github - v2 #11762

Closed
wants to merge 1 commit into from

Conversation

jasonish
Copy link
Member

This allows the source override in .cargo/config.toml to apply to the suricata-lua-sys crate as well fixing offline builds.

Ticket: #7226

Previous PR: #11624

Changes from last PR:

  • Update the lock file as well

This allows the source override in .cargo/config.toml to apply to the
suricata-lua-sys crate as well fixing offline builds.

Ticket: OISF#7226
Copy link

codecov bot commented Sep 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.54%. Comparing base (31bed10) to head (c0fa813).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #11762      +/-   ##
==========================================
- Coverage   82.62%   82.54%   -0.09%     
==========================================
  Files         919      919              
  Lines      248979   248979              
==========================================
- Hits       205722   205517     -205     
- Misses      43257    43462     +205     
Flag Coverage Δ
fuzzcorpus 60.32% <ø> (-0.54%) ⬇️
livemode 18.71% <ø> (+<0.01%) ⬆️
pcap 44.18% <ø> (+0.04%) ⬆️
suricata-verify 61.90% <ø> (+0.01%) ⬆️
unittests 58.99% <ø> (+<0.01%) ⬆️

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

@jasonish jasonish mentioned this pull request Sep 12, 2024
@victorjulien victorjulien added this to the 8.0 milestone Sep 12, 2024
@victorjulien
Copy link
Member

Merged in #11768, thanks!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants