Skip to content

Conversation

@cclauss
Copy link
Contributor

@cclauss cclauss commented Jan 16, 2024

As discussed at #236 (review)

@alxbridge @zerolab Your reviews, please.

Description

Please include a summary of the changes and which issue this relates to (if applicable).
GitHub Actions: Add Python 3.12 to the testing

Fixes # (issue)

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have added an appropriate CHANGELOG entry
Copy link
Member

@zerolab zerolab left a comment

Choose a reason for hiding this comment

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

Thank you for adding these @cclauss!

Left a couple of tidy-up comments which would be nice to address while in this context, if you have the time

Copy link
Member

@zerolab zerolab left a comment

Choose a reason for hiding this comment

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

LGTM!

@zerolab zerolab merged commit 829f7ac into torchbox:main Jan 17, 2024
@cclauss cclauss deleted the patch-1 branch January 17, 2024 12:03
CuriousLearner added a commit to lincolnloop/django-pattern-library that referenced this pull request Feb 24, 2024
…rary into jinja * 'main' of https://github.com/torchbox/django-pattern-library: Add Python 3.12 to the test matrix, drop Django 4.1 (torchbox#242) Updates for version 1.2.0 Improve handling of non-string values for 'override_tag's 'default_html' argument (torchbox#224) fix: make it work for django5.0 (updated) (torchbox#241) Upgrade GitHub Actions versions (torchbox#237) Fix typos discovered by codespell (torchbox#238)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants