Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

Update for Kedro 0.17.0 #12

Merged
merged 4 commits into from
Feb 2, 2021
Merged

Update for Kedro 0.17.0 #12

merged 4 commits into from
Feb 2, 2021

Conversation

lorenabalan
Copy link
Contributor

@lorenabalan lorenabalan commented Feb 1, 2021

Updating this ahead of beginner training this week.

Steps:

kedro new --starter spaceflights

prompts:

kedro_tutorial
spaceflight
kedro_tutorial

Deleted pipeline code from hooks.py, src/pipelines/data_engineering/pipeline.py and src/pipelines/data_science/pipeline.py.

@lorenabalan lorenabalan self-assigned this Feb 1, 2021
@@ -52,14 +54,15 @@
"docs": [
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you sync docs dependencies with kedro code? Some deps are redundant and some need a version update.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think these are in line with the requirements packaged with kedro 0.17.0 at the time, no?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, sorry I missed that it is going to be updated in the future release,

lorenabalan and others added 3 commits February 2, 2021 10:41
Copy link
Contributor

@yetudada yetudada left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🏆

@yetudada yetudada merged commit fcc7cbe into master Feb 2, 2021
@yetudada yetudada deleted the update-0.17.0 branch February 2, 2021 13:17
antonymilne pushed a commit to antonymilne/kedro-training that referenced this pull request Mar 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants