-
Notifications
You must be signed in to change notification settings - Fork 151
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
[Docs] Add kedro_integration to API reference #1020
Conversation
Signed-off-by: Guillaume Tauzin <[email protected]>
Signed-off-by: Guillaume Tauzin <[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 very much for doing this - it's been on my list for ages but I didn't get around to it 🙏 And great timing, I was just updating my kedro docs this afternoon too!
I just suggested a couple of tiny changes but happy to approve. Once this is merged I'll modify #1014 to link to your new page.
vizro-core/changelog.d/20250214_151624_4648633+gtauzin_kedro_api.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Antony Milne <[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.
This looks great! Thank you @gtauzin 🚀
Description
This adds the kedro integration function to the API reference as discussed with @antonymilne in #1001.
Notice
I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":