-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
⬆️ UPGRADE: Autoupdate pre-commit config #41160
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
⬆️ UPGRADE: Autoupdate pre-commit config #41160
Conversation
like the doc-string changes. the rest i guess are fine as long as they are automatic?. cc @jbrockmendel |
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've merged a PR adding a new pyi file this morning. will probably need to merge master
pandas/_libs/algos.pyi
Outdated
def nancorr_spearman( | ||
mat: np.ndarray, # ndarray[float64_t, ndim=2] | ||
minp: int = 1, | ||
) -> np.ndarray: # np.ndarray[np.float64, ndim=2] |
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.
the comment's gone.
pandas/_libs/window/indexers.pyi
Outdated
np.ndarray, # np.ndarray[np.int64] | ||
np.ndarray, # np.ndarray[np.int64] | ||
]: ... | ||
) -> tuple[np.ndarray, np.ndarray,]: ... # np.ndarray[np.int64] # np.ndarray[np.int64] |
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.
not a big deal, but this is marginally less obvious what the comment is referring to
pandas/_libs/tslibs/conversion.pyi
Outdated
]: ... | ||
|
||
|
||
) -> tuple[int, int,]: ... # int64_t |
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.
this is ambiguous what the int64_t comment is referring to
I didn't run anything here, this is just the latest release of |
70aa9e6
to
6a271a2
Compare
6a271a2
to
709a69c
Compare
709a69c
to
ea37243
Compare
Hello @github-actions[bot]! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2021-05-25 08:24:42 UTC |
ea37243
to
de578bd
Compare
leaving the |
Co-authored-by: MarcoGorelli <[email protected]>
Co-authored-by: MarcoGorelli <[email protected]>
Base PullRequest
⬆️ UPGRADE: Autoupdate pre-commit config (#41160)
Command results
Details:
add path
pip install pre-commit
stderr:
pre-commit autoupdate || (exit 0);
pre-commit run -a || (exit 0);
Changed files
Changed 95 files: