-
Notifications
You must be signed in to change notification settings - Fork 873
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
[Bug fix] - Make cookiecutter arguments consistent for all kedro new path #3266
Conversation
Signed-off-by: Nok Chan <nok.lam.chan@quantumblack.com>
Signed-off-by: Nok Lam Chan <nok.lam.chan@quantumblack.com>
Signed-off-by: Nok Chan <nok.lam.chan@quantumblack.com>
Signed-off-by: Nok Chan <nok.lam.chan@quantumblack.com>
Signed-off-by: Nok Chan <nok.lam.chan@quantumblack.com>
Signed-off-by: Nok Chan <nok.lam.chan@quantumblack.com>
I don't think I can merge this PR in short term. @astrojuanlu any chance you may know someone from the cookiecutters team? We have the problem describe in cookiecutter/cookiecutter#1975. It's not end of the world, the tradeoff is
The tests that I have created in this PR should still be useful, but we don't have to rush it now. |
I will close this PR since this is blocked by an upstream The main thing is it will be nice to refactor this using a proper list when choosing a tool/add-ons, but this doesn't affect the logic. |
I see cookiecutter still has activity, it's probably worth trying to contribute a fix upstream. |
Description
Development notes
pyproject.toml
in starters.test_cookiecutter_args
, this catch a bug that we missed. For examplekedro new --addons none
return a''
for cookiecutter args, but[]
forkedro new
-> none (interactive prompt)"Linting, PySpark"
)pytest-mock
becausespy.return_value
requires at least 1.10.0. Note, I think we should upgradepytest-mock
pytest-mock
in templates at all. kedro project doesn't necessary followkedro
's test requirements. (create a new ticket)Tests are failing because we need to propogate the same changes to
kedro-starters
, I have to use_cookiecutter
instead ofcookiecutter
because of cookiecutter/cookiecutter#1975.Notes:
kedro-starters
or use the--checkout
option (it's a temporary hack that still work because we haven't disable using both options andcookiecutter
is fine with it, actually why do we want to block the--checkout
option?)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-by
line 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.md
file