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: Various fixes #12

Merged
merged 24 commits into from
Sep 2, 2024
Merged

fix: Various fixes #12

merged 24 commits into from
Sep 2, 2024

Conversation

LecrisUT
Copy link
Owner

@LecrisUT LecrisUT commented Feb 24, 2024

  • Update cmake_minimum_required
  • Fix windows tests
  • Fix install for shared library. Static is still weird because it requires installing dependencies
  • Change SonarCloud for Qodana
  • Changed coverage test to use gcov
  • Removed the set(CMAKE_BUILD_TYPE). Instead it should be set via preset or manually, or let the system decide. Can interfere with cmake --install and other processes
  • Simplified packit configuration
  • Added renovate as dependency tracker

Closes #11

Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
@codecov-commenter
Copy link

codecov-commenter commented Feb 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (a4f73db) to head (f363d58).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #12   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines            5         4    -1     
=========================================
- Hits             5         4    -1     
Flag Coverage Δ
functional-tests 100.00% <ø> (ø)
unit-tests 50.00% <ø> (-50.00%) ⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@LecrisUT LecrisUT force-pushed the fix/various-issues branch 2 times, most recently from ac93ac9 to 9c0188a Compare February 26, 2024 09:43
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
@LecrisUT LecrisUT force-pushed the fix/various-issues branch 2 times, most recently from ca81ec0 to 57cf44b Compare February 26, 2024 09:55

This comment was marked as outdated.

- Contains fix for Windows DL_PATHS

Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
- Subprojects should only need to find_package the main project
- Use target check to see if the main project was included

Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
@LecrisUT LecrisUT force-pushed the fix/various-issues branch 4 times, most recently from 1a8b84b to 6a4ee2f Compare September 2, 2024 07:51
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
@LecrisUT LecrisUT force-pushed the fix/various-issues branch 2 times, most recently from 717056a to 5de3aaf Compare September 2, 2024 09:05
This should instead be set by the preset

Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
@LecrisUT LecrisUT merged commit ad3fe1f into main Sep 2, 2024
38 of 40 checks passed
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.

CMAKE_BUILD_TYPE
2 participants