Skip to content

Conversation

@pepe5p
Copy link
Owner

@pepe5p pepe5p commented Aug 9, 2025

No description provided.

@pepe5p pepe5p requested a review from Copilot August 9, 2025 13:34
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR refactors project naming conventions to improve consistency across template files. The changes introduce computed variables for different case formats (kebab-case, snake_case, flat_case, title_case) and update all template references to use the appropriate case for each context.

  • Updates template files to use appropriate naming conventions (kebab-case for packages/containers, snake_case for Python modules)
  • Adds computed variables in copier.yaml to automatically generate different case formats from the base project name
  • Removes unused dependency and adds isort configuration for first-party imports

Reviewed Changes

Copilot reviewed 7 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
copier.yaml Adds computed variables for different case formats and updates help text
skeleton/docker-entrypoint.sh.jinja Updates APP export to use kebab-case
skeleton/docker-compose.yml.jinja Updates service name to use snake_case
skeleton/pyproject.toml.jinja Updates package name to kebab-case, code reference to snake_case, removes unused dependency, adds isort config
skeleton/justfile.jinja Updates Python paths and container name to use snake_case
skeleton/README.md.jinja Updates title to title_case and service references to snake_case
justfile Updates test output directory name and adds explicit test path

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@pepe5p pepe5p merged commit 1c083a1 into main Aug 9, 2025
3 checks passed
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.

2 participants