Skip to content

Conversation

bckohan
Copy link
Member

@bckohan bckohan commented Sep 27, 2025

No description provided.

@Copilot Copilot AI review requested due to automatic review settings September 27, 2025 16:47
@bckohan bckohan merged commit 11a9553 into main Sep 27, 2025
48 of 49 checks passed
@bckohan bckohan deleted the dj6 branch September 27, 2025 16:47
Copy link

@Copilot 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 adds early support for Django 6.0 by testing against Django 6.0a1 (alpha 1) and updating the package version to 3.3.2. The changes prepare the library for compatibility with the upcoming Django 6.0 release while maintaining backward compatibility with existing Django versions.

Key changes:

  • Version bump from 3.3.1 to 3.3.2
  • Expanded Django compatibility from <6.0 to <6.1
  • Added CI testing for Django 6.0a1 with appropriate exclusions for unsupported Python versions

Reviewed Changes

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

File Description
src/django_typer/init.py Bumped package version to 3.3.2
pyproject.toml Updated Django dependency range and added Django 6.0 classifier
doc/source/changelog.rst Added changelog entry for v3.3.2 release
.github/workflows/test.yml Added Django 6.0a1 to test matrix with prerelease installation logic

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

run: |
just test-lock "Django~=${{ matrix.django-version }}.0"
just setup ${{ steps.sp.outputs.python-path }}
if [[ "${{ matrix.django-version }}" =~ (a|b|rc) ]]; then
Copy link
Preview

Copilot AI Sep 27, 2025

Choose a reason for hiding this comment

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

The regex pattern (a|b|rc) may not match all possible Django prerelease patterns. Consider using a more comprehensive pattern like [0-9]+(a|b|rc)[0-9]* to ensure it matches specific alpha/beta/rc versions with numbers.

Suggested change
if [[ "${{ matrix.django-version }}" =~ (a|b|rc) ]]; then
if [[ "${{ matrix.django-version }}" =~ [0-9]+(a|b|rc)[0-9]* ]]; then

Copilot uses AI. Check for mistakes.

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.

1 participant