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

Progress bar gbq #22399

Closed
wants to merge 4 commits into from
Closed

Conversation

bagmanas
Copy link

Add variable for progress bar

@pep8speaks
Copy link

pep8speaks commented Aug 17, 2018

Hello @bagmanas! Thanks for updating the PR.

Cheers ! There are no PEP8 issues in this Pull Request. 🍻

Comment last updated on August 17, 2018 at 17:24 Hours UTC

@WillAyd
Copy link
Member

WillAyd commented Aug 17, 2018

Does this reference a particular issue number?

@WillAyd WillAyd added the Needs Info Clarification about behavior needed to assess issue label Aug 17, 2018
@bagmanas
Copy link
Author

No. This is the problem I faced an hour ago.)

@WillAyd
Copy link
Member

WillAyd commented Aug 17, 2018

Ok please be sure to open an issue first before making any code changes. I’m not entirely sure what this is from limited gbq knowledge but tests are required as well

@TomAugspurger
Copy link
Contributor

Do we have a minimum version for pandas-gbq? Looks like this was added in 0.5.

@alimcmaster1
Copy link
Member

#21628 Looks like the relevant changes have been made in this PR. I think we are good to close this one. @WillAyd @TomAugspurger

@WillAyd
Copy link
Member

WillAyd commented Aug 28, 2018

Still need an issue to be opened for this along with the relevant test case(s)

@alimcmaster1
Copy link
Member

Wait I meant the changes this PR addresses, have already been made on master see #21628. So this is a duplicate. Hence we can close this? @WillAyd

@WillAyd
Copy link
Member

WillAyd commented Aug 31, 2018

@alimcmaster1 ah thanks for the link. Looks like it - closing as duplicative

@WillAyd WillAyd closed this Aug 31, 2018
@WillAyd WillAyd added Duplicate Report Duplicate issue or pull request and removed Needs Info Clarification about behavior needed to assess issue labels Aug 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Duplicate Report Duplicate issue or pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants