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

feat: transition to uv and copier #271

Merged
merged 8 commits into from
Feb 13, 2025
Merged

feat: transition to uv and copier #271

merged 8 commits into from
Feb 13, 2025

Conversation

sinopeus
Copy link
Collaborator

@sinopeus sinopeus commented Feb 11, 2025

  • New name once decided
  • Rewrite README to include callout + migration guide from Poetry Cookiecutter to UV + Copier based flow
  • Update pre-gen hooks to announce migration
  • Fix/update CHANGELOG
  • Optionally: add CI script for testing Poetry Cookiecutter template
  • Add semi-support for GitLab CI/CD (i.e. documentation)

Hesitated to mark this as BREAKING CHANGE but in principle we're backwards compatible for those still using Cruft + Cookiecutter – the day we remove the old template altogether will be a breaking change though.

@sinopeus sinopeus requested a review from lsorber February 11, 2025 07:33
@sinopeus sinopeus self-assigned this Feb 11, 2025
This ensures that by the time the Dev Container is created, the shell
prompt is effectively configured. If done in postCreateCommand, the
prompt is only configured after the container's creation and after `uv
sync` and `pre-commit install` are complete.
If the UV cache is not populated, the `uv lock --check --offline`
command will fail as it doesn't just check consistency, it also checks
the cache for the package metadata and fails on a cache miss due to the
`--offline` flag. Removing `--offline` simply means that in the case of
a cache miss, which will in principle be a rare occurrence, we will
fetch the metadata from the package repository,

```
× No solution found when resolving dependencies:
  ╰─▶ Because commitizen was not found in the cache and your project depends
      on commitizen>=4.1.0, we can conclude that your project's requirements
      are unsatisfiable.

      hint: Packages were unavailable because the network was disabled. When
      the network is disabled, registry packages may only be read from the
      cache."
```
@lsorber lsorber changed the title feat: Move to UV + Copier feat: transition to uv and copier Feb 13, 2025
@sinopeus sinopeus marked this pull request as ready for review February 13, 2025 09:44
@sinopeus sinopeus merged commit 2c4905f into main Feb 13, 2025
20 checks passed
@lsorber lsorber deleted the xga-uv-copier branch February 17, 2025 08:26
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.

3 participants