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

improve all ci's #3

Merged
merged 5 commits into from
Oct 3, 2024
Merged

improve all ci's #3

merged 5 commits into from
Oct 3, 2024

Conversation

Totto16
Copy link
Owner

@Totto16 Totto16 commented Oct 2, 2024

  • compress statements, that need cd's , so that they are not done multiple times
  • compress multiple statements, that are only different in one word, use conditions for that

Depends on OpenBrickProtocolFoundation/oopetris#185

compress statements, that need cd's , so that they are not done multiple times
compress multiple statements, that are only different in one word, use conditions for that
Copy link

github-actions bot commented Oct 2, 2024

badge

Code Coverage Summary

Filename                     Stmts    Miss  Cover    Missing
-------------------------  -------  ------  -------  -------------------------------------------------------------------------------------
src/c_wrapper/wrapper.cpp      524     235  55.15%   62-63, 100, 108-109, 179-181, 232, 278, 420, 427-428, 474, 506, 551-552, 566, 587-981
TOTAL                          524     235  55.15%

Results for commit: e1e3c1a

Minimum allowed coverage is 0%

♻️ This comment has been updated with latest results

reorganize the folder structure, so that it can be better used as external library
add windows export declarations
better handle headers
add variables to the pkgconfig file, to resemble oopetris-core
@Totto16 Totto16 merged commit f033ae4 into main Oct 3, 2024
9 checks passed
@Totto16 Totto16 deleted the improve_ci branch October 3, 2024 21:34
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.

1 participant