-
Notifications
You must be signed in to change notification settings - Fork 41
Conversation
… initialize new plugin repository)
Some first thoughts:
|
I pulled this down and played with it a bit. In general I do like it - I like how much more flexibility there is over the prompts. It looks like the I don't like that basically every file in the template ends with I don't know if this is something we can configure on our end but right now you have to use Template uses potentially unsafe features: jinja_extensions, tasks.
If you trust this template, consider adding the `--trust` option when running `copier copy/update`. Also, I needed to manually install Finally, I think currently the |
jinja2-time
I added a line about installing Also, we only use the --trust
This is unavoidable if you want to use jinja extensions, tasks, or migrations, since all three copier features allow the execution of arbitrary (and thus potentially unsafe) code. We can't configure this in the
It's my understanding that the tasks are important to run, but if there's a way to rework that perhaps you can avoid the |
Hmm, that's not great 😦 |
Discussed at the PR party with @GenevieveBuckley, @jni and @psobolewskiPhD. We want to move forward with |
Closing in favour of napari/napari-plugin-template#1 |
This PR swaps copier for cookiecutter.
It is a fully working implementation that people can try out (as suggested by @jni, see context below)
napari/napari#6122 by @jni