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.

Sign up for GitHub

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

Jump to bottom

Bump dataframe/1.15.0 #4051

Merged
merged 3 commits into from
Jan 13, 2021
Merged

Bump dataframe/1.15.0 #4051

merged 3 commits into from
Jan 13, 2021

Conversation

theirix
Copy link
Contributor

@theirix theirix commented Jan 1, 2021

Specify library name and version: dataframe/1.15.0

  • I've read the guidelines for contributing.
  • I've followed the PEP8 style guides for Python code in the recipes.
  • I've used the latest Conan client version.
  • I've tried at least one configuration locally with the
    conan-center hook activated.

Croydon
Croydon previously approved these changes Jan 1, 2021
View reviewed changes
This was referenced Jan 1, 2021
Closed
Closed
@ghost
Copy link

ghost commented Jan 1, 2021

I detected other pull requests that are modifying dataframe/all recipe:

This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there.

This was referenced Jan 1, 2021
Open
Closed
@conan-center-bot
Copy link
Collaborator

An unexpected error happened and has been reported. Help is on its way! 🏇

dmn-star
dmn-star previously approved these changes Jan 2, 2021
View reviewed changes
hosseinmoein
hosseinmoein previously approved these changes Jan 2, 2021
View reviewed changes
@hosseinmoein
Copy link
Contributor

I closed the other conflicting PR

@theirix theirix closed this Jan 4, 2021
@theirix theirix reopened this Jan 4, 2021
@conan-center-bot
Copy link
Collaborator

An unexpected error happened and has been reported. Help is on its way! 🏇

@hosseinmoein
Copy link
Contributor

Is there any resolution as to why this is not building?
Thanks

@hosseinmoein
Copy link
Contributor

hosseinmoein commented Jan 8, 2021 •
edited
Loading

@Croydon , @prince-chrismc , @dmn-star
Is there anything we can do to move this forward?
Thanks

prince-chrismc
prince-chrismc suggested changes Jan 8, 2021
View reviewed changes
recipes/dataframe/all/conanfile.py Outdated Show resolved Hide resolved
@theirix @prince-chrismc
Update recipes/dataframe/all/conanfile.py
68e6275 
Co-authored-by: Chris Mc <prince.chrismc@gmail.com>
@theirix theirix dismissed stale reviews from hosseinmoein, dmn-star, and Croydon via 68e6275 January 8, 2021 14:37
@conan-center-bot
Copy link
Collaborator

An unexpected error happened and has been reported. Help is on its way! 🏇

@prince-chrismc
Copy link
Contributor

You'll need to restart it again

@theirix theirix closed this Jan 8, 2021
@theirix theirix reopened this Jan 8, 2021
@theirix
Copy link
Contributor Author

theirix commented Jan 8, 2021

You'll need to restart it again

didn't help

@prince-chrismc
Copy link
Contributor

Status is in progress... it's doing something now

@conan-center-bot
Copy link
Collaborator

All green in build 4 (68e62758abaef833085f16f086ce854338be7ec9)! 😊

@github-actions github-actions bot mentioned this pull request Jan 8, 2021
Open
@hosseinmoein
Copy link
Contributor

@prince-chrismc , @SpaceIm , @mathbunnyru
Is this mergeable now?
Thanks,

@prince-chrismc
Copy link
Contributor

#2857 Has some details about what's required for merging, but we are missing one for the CCI team...

Please be patient they are doing their best to keep up with all the PRs we get

@mathbunnyru
Copy link
Contributor

Sorry, I'm not official community reviewer, just trying to help the CCI team when I can.

@SSE4 SSE4 requested a review from uilianries January 12, 2021 16:53
@conan-center-bot conan-center-bot merged commit d5efc74 into conan-io:master Jan 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hosseinmoein hosseinmoein hosseinmoein approved these changes

@SpaceIm SpaceIm SpaceIm approved these changes

@prince-chrismc prince-chrismc prince-chrismc approved these changes

@mathbunnyru mathbunnyru mathbunnyru approved these changes

@SSE4 SSE4 SSE4 approved these changes

@dmn-star dmn-star dmn-star left review comments

@Croydon Croydon Croydon left review comments

@uilianries uilianries Awaiting requested review from uilianries

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@theirix @conan-center-bot @hosseinmoein @prince-chrismc @mathbunnyru @SSE4 @Croydon @SpaceIm @dmn-star