Skip to content
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

implement support for local extensions update #254

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Chilipp
Copy link
Contributor

@Chilipp Chilipp commented May 31, 2023

This PR implements the support for an update of local extensions. Once we remove the path from sys.path in the cleanup step, we should also remove all modules that are in this folder from sys.modules

Important notice: Before we can merge this PR, cruft/cookiecutter-test#7 needs to be merged and the temporary change in test_local_extension_update needs to be switched back to the official cruft/cookiecutter-test repo

closes #253

@Chilipp
Copy link
Contributor Author

Chilipp commented May 31, 2023

hmm, no idea why this is failing. When I do it locally, i.e.

cruft create https://github.com/Chilipp/cookiecutter-test --directory dir --checkout extensions -y
cruft update --project-dir test --checkout extensions-update --skip-apply-ask

it sometimes works and sometimes doesn't

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Local extensions cannot be updated
1 participant