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

Add mem leak test #126

Closed
wants to merge 1 commit into from
Closed

Add mem leak test #126

wants to merge 1 commit into from

Conversation

ladisgin
Copy link
Member

@ladisgin ladisgin commented Oct 2, 2023

Summary:

Checklist:

  • The PR addresses a single issue. If it can be divided into multiple independent PRs, please do so.
  • The PR is divided into a logical sequence of commits OR a single commit is sufficient.
  • There are no unnecessary commits (e.g. commits fixing issues in a previous commit in the same PR).
  • Each commit has a meaningful message documenting what it does.
  • All messages added to the codebase, all comments, as well as commit messages are spellchecked.
  • The code is commented OR not applicable/necessary.
  • The patch is formatted via clang-format OR not applicable (if explicitly overridden leave unchecked and explain).
  • There are test cases for the code you added or modified OR no such test cases are required.

@codecov-commenter
Copy link

codecov-commenter commented Oct 2, 2023

Codecov Report

Merging #126 (4db817c) into main (390866f) will increase coverage by 0.00%.
The diff coverage is n/a.

❗ Current head 4db817c differs from pull request most recent head a42bbe1. Consider uploading reports for the commit a42bbe1 to get more accurate results

@@           Coverage Diff           @@
##             main     #126   +/-   ##
=======================================
  Coverage   66.43%   66.43%           
=======================================
  Files         215      215           
  Lines       29636    29636           
  Branches     6572     6572           
=======================================
+ Hits        19688    19690    +2     
+ Misses       7433     7432    -1     
+ Partials     2515     2514    -1     

see 3 files with indirect coverage changes

@ladisgin ladisgin closed this Oct 2, 2023
@ladisgin ladisgin deleted the ladisgin/mem_leak branch October 2, 2023 14:27
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.

2 participants