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

Fix fuzzer build & increase RSS limit #313

Merged
merged 2 commits into from
Nov 7, 2023
Merged

Conversation

estringana
Copy link
Collaborator

Description

Motivation

Additional Notes

Describe how to test your changes

Readiness checklist

  • Unit tests have been updated and pass
  • If known, an appropriate milestone has been selected
  • All new source files include the required notice

Release checklist

  • The CHANGELOG.md has been updated

@estringana estringana closed this Nov 7, 2023
@estringana estringana deleted the estringana/add-missing-header branch November 7, 2023 15:30
@Anilm3 Anilm3 restored the estringana/add-missing-header branch November 7, 2023 15:31
@Anilm3 Anilm3 reopened this Nov 7, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #313 (d03549c) into master (0cfbc29) will decrease coverage by 10.73%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master     #313       +/-   ##
===========================================
- Coverage   64.14%   53.42%   -10.73%     
===========================================
  Files          95       70       -25     
  Lines        6024     2587     -3437     
  Branches     1952     1165      -787     
===========================================
- Hits         3864     1382     -2482     
+ Misses       1008      447      -561     
+ Partials     1152      758      -394     
Flag Coverage Δ
extension ?
helper 53.42% <ø> (ø)

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

Files Coverage Δ
src/helper/remote_config/protocol/client.hpp 100.00% <ø> (ø)

... and 25 files with indirect coverage changes

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@Anilm3 Anilm3 changed the title Add missing header to remote config client Fix fuzzer build & increase RSS limit Nov 7, 2023
@Anilm3 Anilm3 marked this pull request as ready for review November 7, 2023 15:56
@Anilm3 Anilm3 requested a review from a team as a code owner November 7, 2023 15:56
@Anilm3 Anilm3 merged commit caee6c8 into master Nov 7, 2023
24 checks passed
@Anilm3 Anilm3 deleted the estringana/add-missing-header branch November 7, 2023 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants