Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for the [djlinter](https://www.djlint.com/) #4909

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

vds2212
Copy link

@vds2212 vds2212 commented Feb 16, 2025

This code aim to propose a linter for the djlinter from: djlinter

Copy link
Contributor

@hsanson hsanson left a comment

Choose a reason for hiding this comment

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

PR in incomplete. Please ensure to:

  • Add linter tests (example)
  • Add handler tests (example)
  • Add entry to supported-tools.md (example)
  • Add entry to doc/ale.txt (example)
  • Add entry to ale-supported-languages-and-tools.txt (example)
  • Add entry to ale-html.txt (example)
  • Ensure all above entries are added in alphabetical order.

@vds2212
Copy link
Author

vds2212 commented Feb 21, 2025

I have complete the tasks indicated. Let me know if you want me to do something else.

@vds2212 vds2212 requested a review from hsanson February 24, 2025 05:18
@vds2212
Copy link
Author

vds2212 commented Feb 24, 2025

I found an obvious mistake :-|
Is there a way I can run the test myself before requesting GitHub to re-run them?

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