docs: fix broken venv command and broken external link in MLflow integration docs#5394
Merged
SajidAlamQB merged 2 commits intokedro-org:mainfrom Feb 26, 2026
Merged
Conversation
Member
|
Hi @AutoHmn13 , thank you so much for your work on this! It looks like these tasks were very popular and we had several contributors address the sub-tasks of #5271. Would you mind adjusting your PR to address the final remaining task #5383 and the issue you found with the broken |
Contributor
Author
|
No problem @merelcht - adjusted PR coming shortly 😄 |
…gration docs Signed-off-by: AutoHmn13 <shmrck206@gmail.com> Made-with: Cursor
AutoHmn13
force-pushed
the
docs/fix-mlflow-integration-docs
branch
from
ab17ac0 to
8756a97
Compare
AutoHmn13
changed the title
merelcht
approved these changes
Feb 26, 2026
Member
merelcht
left a comment
merelcht
left a comment
There was a problem hiding this comment.
Thanks a lot for the contribution @AutoHmn13 ! ⭐
SajidAlamQB
approved these changes
Feb 26, 2026
Contributor
SajidAlamQB
left a comment
SajidAlamQB
left a comment
There was a problem hiding this comment.
Thank you @AutoHmn13! 👍
Contributor
Author
|
Thanks for the opportunity, it was fun 😄 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…Lflow integration docs
Description
Fixes #5383
Closes #5271 (partial — remaining sub-task)
Two documentation fixes in
docs/integrations-and-plugins/mlflow.md:python -m venvcommand: Setup block was missing the target directory —python -m venv && ...→python -m venv .venv && ...kedro-mlflowdocs #5383): The{external+kedro-mlflow:doc}intersphinx role referenced a non-existent path. Replaced with a direct Markdown link to the correct kedro-mlflow docs page.Development notes
Docs-only change — no code modified, no tests required, no Kedro-Viz impact.
Developer Certificate of Origin
We need all contributions to comply with the Developer Certificate of Origin (DCO). All commits must be signed off by including a
Signed-off-byline in the commit message. See our wiki for guidance.If your PR is blocked due to unsigned commits, then you must follow the instructions under "Rebase the branch" on the GitHub Checks page for your PR. This will retroactively add the sign-off to all unsigned commits and allow the DCO check to pass.
Checklist
RELEASE.mdfile