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

STYLE: Exclude package information file from PEP8 checks #2241

Merged

Conversation

jhlegarreta
Copy link
Contributor

Exclude package information file from PEP8 checks.

Exclude package information file from PEP8 checks.
Copy link
Member

@skoudoro skoudoro left a comment

Choose a reason for hiding this comment

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

Thanks @jhlegarreta. LGTM!

@codecov
Copy link

codecov bot commented Sep 28, 2020

Codecov Report

Merging #2241 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2241      +/-   ##
==========================================
- Coverage   91.37%   91.37%   -0.01%     
==========================================
  Files         252      252              
  Lines       32911    32911              
  Branches     3460     3460              
==========================================
- Hits        30074    30072       -2     
  Misses       2079     2079              
- Partials      758      760       +2     
Impacted Files Coverage Δ
dipy/viz/app.py 76.98% <0.00%> (-0.50%) ⬇️
dipy/reconst/tests/test_mcsd.py 98.82% <0.00%> (ø)

@skoudoro skoudoro merged commit 48915ba into dipy:master Sep 29, 2020
@skoudoro
Copy link
Member

Thank you @jhlegarreta

@jhlegarreta jhlegarreta deleted the ExcludePackageInfoFileFromPEP8Checks branch September 29, 2020 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants