Skip to content

Tests#163

Open
ragusa87 wants to merge 1 commit into
masterfrom
tests
Open

Tests#163
ragusa87 wants to merge 1 commit into
masterfrom
tests

Conversation

@ragusa87

Copy link
Copy Markdown

No description provided.

Copilot AI review requested due to automatic review settings June 19, 2026 10:24
@ragusa87 ragusa87 force-pushed the tests branch 3 times, most recently from 2a3ee28 to 44354b5 Compare June 19, 2026 10:28

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds contributor/operator documentation and updates the template’s baseline dependencies and CI configuration.

Changes:

  • Added AGENTS.md documenting the cookiecutter template structure and how to locally validate template changes.
  • Updated the generated project’s base requirements to target Django >=5.2,<6.0.
  • Modified the repo root .gitlab-ci.yml to use a container image and a new test entrypoint script.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

File Description
AGENTS.md New repo guide explaining layout, generated project contents, and manual validation steps.
{{cookiecutter.project_slug}}/requirements/base.in Bumps the generated project’s Django requirement range.
.gitlab-ci.yml Updates the template repo’s GitLab CI job image and test command.
Comments suppressed due to low confidence (1)

.gitlab-ci.yml:3

  • .gitlab-ci.yml runs ./hooks/test.sh, but there is no hooks/test.sh in this repo (the hooks/ dir only contains post_gen_project.py). This will make the GitLab pipeline fail immediately. Consider inlining the template smoke-test (generate with cookiecutter, install generated project's test deps, run scripts/run_tests.sh) and use the same Python version as the template (3.11) for consistency.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ragusa87 ragusa87 changed the base branch from master to lts June 19, 2026 12:15
@ragusa87 ragusa87 changed the base branch from lts to master June 19, 2026 12:23
@ragusa87 ragusa87 requested a review from Copilot June 19, 2026 12:24

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

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

Comment on lines +8 to +13
tests:
runs-on: ubuntu-latest
container: ghcr.io/astral-sh/uv:python3.14-trixie
steps:
- uses: actions/checkout@v4
- run: ./hooks/test.sh
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