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/602/20241015/v1 #11965

Merged
merged 8 commits into from
Oct 16, 2024
Merged

Conversation

AkakiAlice and others added 8 commits October 15, 2024 13:54
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4.4.0 to 4.4.3.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@5076954...b4b15b8)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3.26.10 to 3.26.12.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Commits](github/codeql-action@v3.26.10...v3.26.12)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [actions/checkout](https://github.com/actions/checkout) from 4.2.0 to 4.2.1.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@d632683...eef6144)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
This was done following the fact that this setting was historically
named incorrectly. The purpose of the setting was always to define the
ports that will be prioritized and have rule groups associated w them on
priority. Rename all occurences of this to correctly reflect the purpose
of the setting.
@victorjulien victorjulien requested review from jasonish and a team as code owners October 15, 2024 14:14
@catenacyber
Copy link
Contributor

#11959 changes a configuration key without documenting it, is it ok ?

@victorjulien
Copy link
Member Author

#11959 changes a configuration key without documenting it, is it ok ?

Yes, in this case. Replied in the PR. Thanks for catching this btw.

Copy link

codecov bot commented Oct 15, 2024

Codecov Report

Attention: Patch coverage is 75.00000% with 14 lines in your changes missing coverage. Please review.

Project coverage is 82.77%. Comparing base (378f678) to head (37fa2a6).
Report is 8 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #11965      +/-   ##
==========================================
+ Coverage   82.76%   82.77%   +0.01%     
==========================================
  Files         910      910              
  Lines      249014   249016       +2     
==========================================
+ Hits       206105   206134      +29     
+ Misses      42909    42882      -27     
Flag Coverage Δ
fuzzcorpus 60.77% <62.50%> (-0.01%) ⬇️
livemode 18.71% <46.42%> (+<0.01%) ⬆️
pcap 44.08% <64.28%> (+0.23%) ⬆️
suricata-verify 62.19% <75.00%> (+0.02%) ⬆️
unittests 59.01% <55.35%> (+<0.01%) ⬆️

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

@suricata-qa
Copy link

Information: QA ran without warnings.

Pipeline 23108

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

Successfully merging this pull request may close these issues.

6 participants