Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@chadac
Copy link
Contributor

@chadac chadac commented Jun 13, 2022

Description

Resolves: #93

Unfortunately there aren't any ways to generate multiple PR templates, so title conventions may not be automatic.

Pull Request Checklist

  • Added tests for added or changed code.
  • Added documentation for changed code.

@codecov
Copy link

codecov bot commented Jun 13, 2022

Codecov Report

Merging #99 (79c2f14) into master (b0ed09d) will increase coverage by 1.70%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master     #99      +/-   ##
=========================================
+ Coverage    0.24%   1.95%   +1.70%     
=========================================
  Files          10       9       -1     
  Lines        2047    2049       +2     
  Branches      487     488       +1     
=========================================
+ Hits            5      40      +35     
+ Misses       2042    2009      -33     
Flag Coverage Δ
unittests 1.95% <ø> (+1.70%) ⬆️

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

Impacted Files Coverage Δ
src/slickml/metrics.py 0.00% <0.00%> (ø)
src/slickml/plotting.py 0.00% <0.00%> (ø)
src/slickml/utilities.py 0.00% <0.00%> (ø)
src/slickml/regression.py 0.00% <0.00%> (ø)
src/slickml/classification.py 0.00% <0.00%> (ø)
src/slickml/feature_selection.py 0.00% <0.00%> (ø)
src/slickml/feature_engineering.py
src/slickml/_dummy.py
src/slickml/utils/_transform.py 100.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b0ed09d...79c2f14. Read the comment docs.

@amirhessam88 amirhessam88 self-requested a review June 13, 2022 15:11
Copy link
Member

@amirhessam88 amirhessam88 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@chadac chadac merged commit 87d5aa8 into slickml:master Jun 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[DOCUMENTATION]: Create Pull Request Template

2 participants