Conversation
johndonor3
left a comment
There was a problem hiding this comment.
I don't have any strong opinions here. It seems simpler than the previous template though so that's good!
|
I'll take a look at this soon. Maybe by next week's coding time. Regarding |
|
Yes, it seems people still commit to |
|
I'm testing this out. When I run |
|
I think that's because it's defaulting to the |
|
That worked. I'm surprised it didn't work when it was pointed to the local branch checkout. One quick question on the uv venv setup: What determines which python version it creates the venv for? It defaults to 12, whether I created it from a different env or not. Are we able to have a question asking which python version to use? I'll keep playing around with this. |
|
I think to run it from a local checkout you need to do something like Yes, we could ask a question about the |
|
Yesterday you mentioned being able to update the template for already generated project. I think that's a good idea. |
havok2063
left a comment
There was a problem hiding this comment.
This is looking good to me. The output project is nice and clean. One thought I had was adding a question to optionally create a minimal Dockerfile for the package.
Also, I wonder if it's worth thinking about a migration script to help packages move from v1->v2->v3, etc, or future versions. I'm not sure how feasible that is. Perhaps it's easier moving forward, after projects are made with copier.
| [](https://www.gnu.org/licenses/gpl-3.0) | ||
| {% endif %} | ||
| <!-- [](https://{{ project_name }}.readthedocs.io/en/latest/) --> | ||
| <!-- [](https://codecov.io/gh/{{github_organization}}/{{ project_name }}) --> |
There was a problem hiding this comment.
Can we add some boilerplate instructions to this readme for installation, dev installation, and how to serve the docs locally?
There was a problem hiding this comment.
Done. Can you check the text? I mostly copied what you added for sdssdb.
|
Yeah I think "project updates" would be good to include. Besides supporting package/project updates, it would allow people to re-answer optional questions for adding features. We might also want a separate script/repo for pushing mass updates to multiple packages. |
|
I included a question to add a I like the idea of allowing to update projects generated with the template. I will probably require a bit of testing and playing around with the options. I think an automated conversion tool from v1 and v2 may be difficult to write since those versions had some many forking paths, but maybe we can include some documentation on how to do it manually. |
I have done some updating of both the template and the style guide, which were over three years old.
This new template (v3) uses copier because
cookiecutterseems to have become abandonware. I have remove most of the configuration options and settled for using uv, Ruff, and furo for Sphinx theming since I think that's what we all are using these days and it simplifies the template and the documentation.The new version of the documentation can be previewed here and the style guide here.
This probably deserves a synchronous code-review.
Closes #25