Skip to content

[MNT] add sklearn to test dependency set#172

Merged
fkiraly merged 2 commits into
mainfrom
add-sklearn-to-tests
May 2, 2023
Merged

[MNT] add sklearn to test dependency set#172
fkiraly merged 2 commits into
mainfrom
add-sklearn-to-tests

Conversation

@fkiraly
Copy link
Copy Markdown
Contributor

@fkiraly fkiraly commented May 2, 2023

This explicitly adds sklearn to tests dependency set, as it is required in some tests for compatibility.

@fkiraly fkiraly added the maintenance Continuous integration, unit testing & package distribution label May 2, 2023
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 2, 2023

Codecov Report

Merging #172 (a2f618b) into main (c94e769) will increase coverage by 0.43%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main     #172      +/-   ##
==========================================
+ Coverage   82.46%   82.89%   +0.43%     
==========================================
  Files          39       39              
  Lines        2777     2777              
==========================================
+ Hits         2290     2302      +12     
+ Misses        487      475      -12     

see 2 files with indirect coverage changes

@fkiraly fkiraly changed the title [MNT] add sklearn to tests dependency set [MNT] add sklearn to test dependency set May 2, 2023
@fkiraly fkiraly merged commit afb291a into main May 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Continuous integration, unit testing & package distribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants