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

Skip to content

Conversation

@amirhessam88
Copy link
Member

  • Updated the dependencies version in pyproject.toml; this would help us to main the package easier with user wider syntax for versions.
  • Updated poetry.lock file.
  • Added flake8-comma plugin; Therefore, flake8 now captures C812 which is the trailing commas. Please note that black does not solve this issue 100%.
  • Added add-trailing-comma to help black in this regards.
  • Updated poe format command with add-trailing-comma.
  • Updated CONTRIBUTING.md document with poetry env use command.

@codecov
Copy link

codecov bot commented Jun 12, 2022

Codecov Report

Merging #95 (13c1515) into master (1e9ae11) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #95   +/-   ##
======================================
  Coverage    0.24%   0.24%           
======================================
  Files          10      10           
  Lines        2047    2047           
  Branches      487     487           
======================================
  Hits            5       5           
  Misses       2042    2042           
Flag Coverage Δ
unittests 0.24% <ø> (ø)

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

Impacted Files Coverage Δ
src/slickml/classification.py 0.00% <ø> (ø)
src/slickml/feature_selection.py 0.00% <ø> (ø)
src/slickml/metrics.py 0.00% <ø> (ø)
src/slickml/plotting.py 0.00% <ø> (ø)
src/slickml/regression.py 0.00% <ø> (ø)
src/slickml/utilities.py 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 1e9ae11...13c1515. Read the comment docs.

@amirhessam88 amirhessam88 merged commit 23446d3 into master Jun 12, 2022
@amirhessam88 amirhessam88 deleted the amir/trailing-comma branch June 12, 2022 02:25
@amirhessam88 amirhessam88 self-assigned this Dec 26, 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.

2 participants