-
Notifications
You must be signed in to change notification settings - Fork 909
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
Improve Documentation for Custom Starters in kedro new (#2618) #3771
Conversation
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 this PR @doxenix ! Left a couple of minor comments
@doxenix There seem to be commits from other branches here, do you mind squashing everything and rebasing on top of Also remember to signoff your commit so the DCO check passes. If you click on it you will find instructions. If it was only one commit you can do And if you keep working on this branch, please use the |
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.
Thank you @doxenix ! Feel free to add your name under https://github.com/doxenix/kedro/blob/fix/2618/RELEASE.md#community-contributions
2bfa808
to
dc384f7
Compare
Signed-off-by: Dominik Zalewski <[email protected]>
Signed-off-by: Dominik Zalewski <[email protected]>
Signed-off-by: Juan Luis Cano Rodríguez <[email protected]>
Signed-off-by: Merel Theisen <[email protected]>
Signed-off-by: Merel Theisen <[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 a lot for this contribution @doxenix ⭐
Subject:
This pull request aims to enhance the user experience and clarity of documentation within kedro new command for custom starters. Specifically, it addresses issue #2618