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

Reduce warnings from CMake #334

Merged
merged 1 commit into from
Aug 9, 2023
Merged

Reduce warnings from CMake #334

merged 1 commit into from
Aug 9, 2023

Conversation

nrkramer
Copy link
Collaborator

Fixes the syntax on these lines so that CMake doesn't warn us about commands with parameters next to eachother.

@github-actions github-actions bot added the test Something about test label Jul 29, 2023
@nrkramer nrkramer self-assigned this Jul 29, 2023
@ghost
Copy link

ghost commented Jul 29, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@codecov
Copy link

codecov bot commented Jul 29, 2023

Codecov Report

Merging #334 (6ab1c90) into master (1b4ed69) will not change coverage.
Report is 1 commits behind head on master.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #334   +/-   ##
=======================================
  Coverage   97.25%   97.25%           
=======================================
  Files          54       54           
  Lines        7977     7977           
=======================================
  Hits         7758     7758           
  Misses        219      219           
Flag Coverage Δ
unittests 97.25% <ø> (ø)

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

@nrkramer nrkramer requested a review from ZigRazor July 29, 2023 19:54
@nrkramer nrkramer requested review from sbaldu and ZigRazor and removed request for ZigRazor and sbaldu August 8, 2023 03:39
@ZigRazor ZigRazor merged commit 530be52 into ZigRazor:master Aug 9, 2023
25 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Something about test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants