-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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/605/70x/20241017/v1 #11982
Merged
victorjulien
merged 16 commits into
OISF:main-7.0.x
from
victorjulien:next/605/70x/20241017/v1
Oct 17, 2024
Merged
next/605/70x/20241017/v1 #11982
victorjulien
merged 16 commits into
OISF:main-7.0.x
from
victorjulien:next/605/70x/20241017/v1
Oct 17, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Ticket: 7309 Do not use a constant expression in a condition (cherry picked from commit 76527dd)
With the latest brew changes, a virtualenv is required to install pyyaml. (cherry picked from commit 2b16369)
(cherry picked from commit a5e13d0)
Currently cbindgen from system packages is broken, for now use the cbindgen artifact we build. (cherry picked from commit 09d604f)
Modification by Jason Ish: Keep removal of duplicate checkout action while resolving conflict. (cherry picked from commit d574d88)
(cherry picked from commit 97d525d)
Mainly hashes of actions and other minor changes.
Sync with master on Fedora builds.
As cocci was removed from the Fedora builds due to issues in Cocci on Fedora.
In job: ubuntu-22-04-cov-fuzz
Pin actions missed as part of the sync with master as they most likely don't exist in master.
Required for the newer version of download-artifact in use due to sync with master.
Not needed as this is a build from make dist, and providing cbindgen will force regeneration of the bindings.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main-7.0.x #11982 +/- ##
=============================================
Coverage ? 82.93%
=============================================
Files ? 922
Lines ? 251298
Branches ? 0
=============================================
Hits ? 208418
Misses ? 42880
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. |
jasonish
approved these changes
Oct 17, 2024
Information: QA ran without warnings. Pipeline 23127 |
This was referenced Oct 17, 2024
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Staging:
SV_BRANCH=OISF/suricata-verify#2100