-
Notifications
You must be signed in to change notification settings - Fork 961
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
fix: bump pandas version limit #2254
Conversation
Signed-off-by: cmuhao <[email protected]>
Congratulations on your first Pull Request and welcome to Amundsen community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/amundsen-io/amundsen/blob/main/CONTRIBUTING.md) |
hi @HaoXuAI thanks for the PR, I approved the checks to run, but based on the failures it looks like the |
Signed-off-by: cmuhao <[email protected]>
@kristenarmes thanks. trying to bump the version for feast |
Signed-off-by: cmuhao <[email protected]>
Signed-off-by: cmuhao <[email protected]>
Signed-off-by: cmuhao <[email protected]>
Signed-off-by: cmuhao <[email protected]>
Signed-off-by: cmuhao <[email protected]>
Signed-off-by: cmuhao <[email protected]>
Signed-off-by: cmuhao <[email protected]>
Signed-off-by: cmuhao <[email protected]>
Signed-off-by: cmuhao <[email protected]>
Signed-off-by: cmuhao <[email protected]>
Signed-off-by: cmuhao <[email protected]>
Signed-off-by: cmuhao <[email protected]>
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 for the persistence in fixing the errors, LGTM
Awesome work, congrats on your first merged pull request! |
@kristenarmes thanks! When would you cut a release to make the change alive? |
let's do |
sounds good. updated here: #2256 |
Description
Motivation and Context
Follow up on PR: #1858, this bump the pandas upper limit to 2.2.2.
How Has This Been Tested?
Documentation
CheckList