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

Add hatch-vcs? #243

Closed
Saransh-cpp opened this issue Jul 26, 2023 · 2 comments · Fixed by #251
Closed

Add hatch-vcs? #243

Saransh-cpp opened this issue Jul 26, 2023 · 2 comments · Fixed by #251

Comments

@Saransh-cpp
Copy link
Contributor

Saransh-cpp commented Jul 26, 2023

I have been using cookie for hatch projects recently, but I could not find an option to include hatch-vcs using the CLI. The setuptools backend includes setuptools_scm by default, should the hatchling backend include hatch-vcs by default too or is it not included intentionally? Thanks!

Edit: hatch-vcs is mentioned in the guide, but not included in cookie.

@henryiii
Copy link
Collaborator

I've actually partially started on adding a true/false choice, vcs. It just turned out to be quite a bit of work to add it to all backends at once, especially some that don't support it well or at all. I removed trampolim a few days ago to help, and I might drop whey. The problem currently would be meson-python; I'd like to get dynamic-metadata finished so we can try adding it there.

Cookiecutter supports bools now, and in copier, it can be really nice with when:. I'm thinking about proposing __when__ to cookiecutter. Then I could just hide the option when it's not supported. :)

@Saransh-cpp
Copy link
Contributor Author

Ah, good to know this is already being worked upon :)

I see, not every backend has a corresponding vcs plugin or library. The last time I was here, the only method to run cookie locally was cookiecutter, but everything has evolved a lot now.

Let me know if I can help in any way! Feel free to close this issue, or generalise it for every backend.

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 a pull request may close this issue.

2 participants