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

change xarray.tests to xarray.testing, and copy xarray.tests.raise_if… #422

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

weipeng1999
Copy link

…_dask_computes into xskillscore.tests.test_skipna_functionality.py

Description

In tests, change xarray.tests to xarray.testing, and copy xarray.tests.raise_if_dask_computes into xskillscore.tests.test_skipna_functionality.py

Pre-Merge Checklist (final steps)

  • I have rebased onto main or develop (wherever I am merging) and dealt with any conflicts.
  • I have squashed commits to a reasonable amount, and force-pushed the squashed commits.

…_dask_computes into xskillscore.tests.test_skipna_functionality.py
@pep8speaks
Copy link

Hello @weipeng1999! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 52:80: E501 line too long (82 > 79 characters)

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.

2 participants