-
-
Notifications
You must be signed in to change notification settings - Fork 18.6k
TST: show_versions test with unmerged commits #32139
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
Conversation
Closing pending investigation in #32120. |
Re-opened, the issue wasnt with un-merged commits, but with git worktree. re-using versioneer code handles the problem. |
gentle ping @simonjayhawkins @WillAyd ; this is workflow-affecting |
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.
@jbrockmendel lgtm
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.
@jbrockmendel lgtm
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.
Quick question but generally lgtm
import pandas as pd | ||
|
||
|
||
@pytest.mark.filterwarnings( |
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.
Are these actually required as part of this change? The linkage here with the rest of the PR isn’t totally clear
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.
these are just because its annoying to have in the output
OK thanks @jbrockmendel Heads up - I’m at CZI this week so may be slow to respond to some of your other PRs. Should catch up over the weekend at the latest |
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff