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

fix(typehint): use redundant alias imports for mypy #167

Closed
wants to merge 3 commits into from
Closed

fix(typehint): use redundant alias imports for mypy #167

wants to merge 3 commits into from

Conversation

adam-grant-hendry
Copy link
Contributor

This commit also better aligns the repo with pyvista/CONTRIBUTING.md.

Fixes Issue #163

This commit also better aligns the repo with `pyvista/CONTRIBUTING.md`.

Fixes Issue #163
@codecov
Copy link

codecov bot commented Jun 26, 2022

Codecov Report

Merging #167 (faeeaa8) into main (9334f9d) will decrease coverage by 0.28%.
The diff coverage is 96.03%.

❗ Current head faeeaa8 differs from pull request most recent head 138c817. Consider uploading reports for the commit 138c817 to get more accurate results

@@            Coverage Diff             @@
##             main     #167      +/-   ##
==========================================
- Coverage   97.22%   96.94%   -0.29%     
==========================================
  Files           8        8              
  Lines         648      654       +6     
  Branches       90       90              
==========================================
+ Hits          630      634       +4     
- Partials       18       20       +2     

Copy link
Member

@tkoyama010 tkoyama010 left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution.

.gitignore Outdated
=======
# virtual environments
.venv/
>>>>>>> Stashed changes
Copy link
Member

Choose a reason for hiding this comment

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

Conflict seems not solved.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ya, nearly all the tests are failing for me in this repo and several simply do not run:

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think this will need some TLC ❤️

… back

Editor-specific files should be ignored in global settings. Certain `ini` files are still used in the `Makefile`, so should be kept.
@adam-grant-hendry
Copy link
Contributor Author

I need to do some cleanup (something went wrong when I forked the repo). I will close this and open a new PR.

@adam-grant-hendry adam-grant-hendry deleted the maint/pytyped branch June 26, 2022 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants