Skip to content

Consolidate flake8 and pydocstyle configuration - #11260

Merged
rapids-bot[bot] merged 4 commits into
NVIDIA:branch-22.08from
vyasr:refactor/consolidate_pydocstyle
Jul 14, 2022
Merged

Consolidate flake8 and pydocstyle configuration#11260
rapids-bot[bot] merged 4 commits into
NVIDIA:branch-22.08from
vyasr:refactor/consolidate_pydocstyle

Conversation

@vyasr

@vyasr vyasr commented Jul 13, 2022

Copy link
Copy Markdown
Contributor

This PR moves all flake8 (both Python and Cython) and pydocstyle configuration into the top-level setup.cfg file. Contributes to #10758.

@vyasr vyasr added 3 - Ready for Review Ready for review by team code quality Python Affects Python cuDF API. improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jul 13, 2022
@vyasr vyasr added this to the CuDF Python Refactoring milestone Jul 13, 2022
@vyasr vyasr self-assigned this Jul 13, 2022
@vyasr
vyasr requested a review from a team as a code owner July 13, 2022 22:14
@vyasr
vyasr requested review from shwina and skirui-source July 13, 2022 22:14
@vyasr vyasr removed this from the CuDF Python Refactoring milestone Jul 13, 2022
@github-actions github-actions Bot removed the Python Affects Python cuDF API. label Jul 13, 2022
@vyasr
vyasr force-pushed the refactor/consolidate_pydocstyle branch from 9cff954 to 979ec9b Compare July 13, 2022 22:15
@vyasr vyasr added this to the CuDF Python Refactoring milestone Jul 13, 2022
@codecov

codecov Bot commented Jul 14, 2022

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (branch-22.08@5393dbf). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             branch-22.08   #11260   +/-   ##
===============================================
  Coverage                ?   86.31%           
===============================================
  Files                   ?      144           
  Lines                   ?    22748           
  Branches                ?        0           
===============================================
  Hits                    ?    19636           
  Misses                  ?     3112           
  Partials                ?        0           

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 5393dbf...979ec9b. Read the comment docs.

@vyasr

vyasr commented Jul 14, 2022

Copy link
Copy Markdown
Contributor Author

@gpucibot merge

@rapids-bot
rapids-bot Bot merged commit 29918e3 into NVIDIA:branch-22.08 Jul 14, 2022
@vyasr
vyasr deleted the refactor/consolidate_pydocstyle branch July 14, 2022 01:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 - Ready for Review Ready for review by team improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants