-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
CI: Update version of 'black' #36493
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
Changes from 8 commits
dbf17ff
4a32460
30f2a59
6259ded
ebbef30
4c67eb7
10eefa5
518f28b
c424404
b73d66a
7c019fd
5fd78c0
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -193,9 +193,7 @@ def test_constructor_inferred_fill_value(self, data, fill_value): | |
assert result == fill_value | ||
|
||
@pytest.mark.parametrize("format", ["coo", "csc", "csr"]) | ||
@pytest.mark.parametrize( | ||
"size", [0, 10], | ||
) | ||
@pytest.mark.parametrize("size", [0, 10]) | ||
@td.skip_if_no_scipy | ||
def test_from_spmatrix(self, size, format): | ||
import scipy.sparse | ||
|
@@ -696,14 +694,23 @@ def test_getslice_tuple(self): | |
res = sparse[ | ||
4:, | ||
] # noqa: E231 | ||
exp = SparseArray(dense[4:,]) # noqa: E231 | ||
exp = SparseArray( | ||
dense[ | ||
4:, | ||
] | ||
) # noqa: E231 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can the trailing comma be removed here? The explicit ignoring of There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Since |
||
tm.assert_sp_array_equal(res, exp) | ||
|
||
sparse = SparseArray(dense, fill_value=0) | ||
res = sparse[ | ||
4:, | ||
] # noqa: E231 | ||
exp = SparseArray(dense[4:,], fill_value=0) # noqa: E231 | ||
exp = SparseArray( | ||
dense[ | ||
4:, | ||
], | ||
fill_value=0, | ||
) # noqa: E231 | ||
tm.assert_sp_array_equal(res, exp) | ||
|
||
msg = "too many indices for array" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks 👍 that solves the typing validation
I'll @ pandas/core / pandas/triage , do you think the rest of the contributors need alerting? I don't think there's that many other people who regularly submit PRs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think pandas/core and pandas/triage is probably fine