Skip to content

test(C): do not use assert#175

Merged
TyBalduf merged 1 commit intodftd4:mainfrom
e-kwsm:assert
Oct 12, 2022
Merged

test(C): do not use assert#175
TyBalduf merged 1 commit intodftd4:mainfrom
e-kwsm:assert

Conversation

@e-kwsm
Copy link
Copy Markdown
Contributor

@e-kwsm e-kwsm commented Oct 6, 2022

assert does nothing if NDEBUG is defined.

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 6, 2022

Codecov Report

Merging #175 (c137190) into main (9ff1dc3) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #175   +/-   ##
=======================================
  Coverage   60.12%   60.12%           
=======================================
  Files          44       44           
  Lines        3130     3130           
  Branches     1019     1019           
=======================================
  Hits         1882     1882           
  Misses        667      667           
  Partials      581      581           
Impacted Files Coverage Δ
test/api/example.c 51.21% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

assert does nothing if NDEBUG is defined
@TyBalduf TyBalduf merged commit 5fe5d11 into dftd4:main Oct 12, 2022
@e-kwsm e-kwsm deleted the assert branch October 12, 2022 22:20
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