-
Notifications
You must be signed in to change notification settings - Fork 93
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
build(datasets): Release kedro-datasets
1.7.0
#326
Conversation
Signed-off-by: Ankita Katiyar <[email protected]>
I'm glad to see the RTD build is catching the indentation error for the polars dataset, but kind of strange it didn't flag when that was merged? 🤔 @astrojuanlu |
Signed-off-by: Ankita Katiyar <[email protected]>
It wasn't added to the |
Signed-off-by: Ankita Katiyar <[email protected]>
Signed-off-by: Ankita Katiyar <[email protected]>
Ugh, exactly, sorry about that. Should we try to make it automatic? Or at least add a checklist item in the pull request template. |
Signed-off-by: Ankita Katiyar <[email protected]>
Co-authored-by: Sajid Alam <[email protected]>
Finally fixed the docstring! Please review, @merelcht @astrojuanlu @SajidAlamQB! |
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 a lot for amending the Polars docstring 🙏🏽 may I ask to cherry-pick #324 for 1.7.0?
## Community contributions | ||
Many thanks to the following Kedroids for contributing PRs to this release: | ||
* [Walber Moreira](https://github.com/wmoreiraa) |
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.
❤️
Signed-off-by: Ankita Katiyar <[email protected]>
Description
Release
kedro-datasets
1.7.0Development notes
Checklist
RELEASE.md
file