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

Put test dependencies in individual projects #361

Merged
merged 5 commits into from
Jul 16, 2024
Merged

Put test dependencies in individual projects #361

merged 5 commits into from
Jul 16, 2024

Conversation

gdalle
Copy link
Member

@gdalle gdalle commented Jul 16, 2024

DI tests

  • Stop parsing dependencies from folder names, put them in individual Project.toml files
  • Put the test env in the LOAD_PATH first

DI CI

  • Adjust names of groups

@codecov-commenter
Copy link

codecov-commenter commented Jul 16, 2024

Codecov Report

Attention: Patch coverage is 79.16667% with 5 lines in your changes missing coverage. Please review.

Project coverage is 96.57%. Comparing base (0db3be8) to head (438496c).

Files Patch % Lines
DifferentiationInterface/test/runtests.jl 70.58% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #361      +/-   ##
==========================================
- Coverage   96.91%   96.57%   -0.34%     
==========================================
  Files         101       98       -3     
  Lines        4824     4813      -11     
==========================================
- Hits         4675     4648      -27     
- Misses        149      165      +16     

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

@gdalle gdalle added the test Related to the testing subpackage label Jul 16, 2024
@gdalle gdalle merged commit 63fa856 into main Jul 16, 2024
51 checks passed
@gdalle gdalle deleted the gd/test_projects branch July 16, 2024 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Related to the testing subpackage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants