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

[Feature Request] Cruft Compatibility #404

Open
GatlenCulp opened this issue Dec 16, 2024 · 3 comments
Open

[Feature Request] Cruft Compatibility #404

GatlenCulp opened this issue Dec 16, 2024 · 3 comments

Comments

@GatlenCulp
Copy link

GatlenCulp commented Dec 16, 2024

image

Cruft is a simple CookieCutter template management system. Including support for it in CCDS would (correct me if I'm wrong) allow users to pull changes from the CCDS template into their pre-existing CCDS-instantiated project. This happens by saving the initial CookieCutter arguments in a json file when the project is instantiated (say with ccds --cruft) and when running cruft update (say ccds --cruft-update) the new template is ran through CookieCutter with the cached arguments, and merging the new instantiation with the actual project with Git.

For example: CCDS V2.1 is released and contains some updated default tooling some user would like to use on their project that they already used CCDS V2.0 to create. By running something like ccds --cruft-update, the user can pull these changes without having to either add these changes by hand or re-instantiate another CCDS project.

As someone who is making their own CCDS fork, I would really like to see this added since I plan on iterating on my template often and one of the major reasons against using CCDS as a starting point was because of this lack of cruft-compatibility.

(Update: I realize this might be better as a discussion rather than issue)

@pjbull
Copy link
Member

pjbull commented Dec 17, 2024

I think it probably makes more sense for cruft to support ccds than the other way around (since ccds is basically cookiecutter with some adjustments for features we needed). Because we need these changes, we can't in the near term make CCDS strictly cookiecutter compatible, and it looks like running cruft probably depends on that.

Have you raised an issue there?

@GatlenCulp
Copy link
Author

I haven't. I might make one soon, if so I will link it here

@GatlenCulp
Copy link
Author

@pjbull I went ahead and wrote an issue on Cruft: cruft/cruft#332

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

No branches or pull requests

2 participants