Skip to content

Conversation

@aNsHuL5217
Copy link

Summary

This PR adds the djade linter to the project's pre-commit configuration and applies formatting fixes to existing Django templates. This ensures consistent code style (e.g., standardizing whitespace in {{ variables }}) across the codebase.

Rationale

As discussed in #2372, djade was selected over alternatives like djLint because djade is actively maintained and specifically targets Django template formatting. This aligns with the project's goal of using stable, long-term tools.

Changes

  • Configuration: Added adamchainz/djade-pre-commit (v1.6.0) to .pre-commit-config.yaml.
  • Formatting: Ran the linter across all files, which automatically updated inconsistent spacing in .html templates.

Verification

  • Ran pre-commit run djade --all-files locally (passed formatting checks).

@aNsHuL5217 aNsHuL5217 marked this pull request as ready for review December 8, 2025 18:27
Copy link
Contributor

@sarahboyce sarahboyce 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! A couple of minor nitpicks

aNsHuL5217 and others added 3 commits December 9, 2025 11:08
Co-authored-by: Sarah Boyce <42296566+sarahboyce@users.noreply.github.com>
Co-authored-by: Sarah Boyce <42296566+sarahboyce@users.noreply.github.com>
Co-authored-by: Sarah Boyce <42296566+sarahboyce@users.noreply.github.com>
Copy link
Contributor

@sarahboyce sarahboyce left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@aNsHuL5217
Copy link
Author

Thanks for the review and approval, @sarahboyce! I appreciate the guidance on the configuration details.

I’ll leave the commits as they are for now, assuming they will be squashed upon merging. Ready when you are!

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