Skip to content

Conversation

LilyFirefly
Copy link
Contributor

@LilyFirefly LilyFirefly commented Sep 15, 2024

Trac ticket number

N/A

Branch description

From the syntax for {{not a var}}, it's clear that %{not a block}% should actually use real block syntax {%not a block%} so we're testing that this is actually handled correctly within a comment.

Checklist

  • This PR targets the main branch.
  • The commit message is written in past tense, mentions the ticket number, and ends with a period.
  • I have checked the "Has patch" ticket flag in the Trac system.
  • I have added or updated relevant tests.
  • I have added or updated relevant docs, including release notes if applicable.
  • I have attached screenshots in both light and dark modes for any UI changes.

From the syntax for `{{not a var}}`, it's clear that `%{not a block}%` should actually use real block syntax `{%not a block%}` so we're testing that this is actually handled correctly within a comment.
@felixxm felixxm changed the title Fix syntax for block within a comment in template test Corrected syntax for block within a comment in template test. Sep 16, 2024
Copy link
Member

@felixxm felixxm left a comment

Choose a reason for hiding this comment

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

@LilyFoote Thanks 👍

Copy link
Contributor

@nessita nessita left a comment

Choose a reason for hiding this comment

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

Makes sense, thank you @LilyFoote! 🌟

@nessita nessita changed the title Corrected syntax for block within a comment in template test. Improved test for TokenType.COMMENT by using correct block syntax in template tests. Sep 16, 2024
@nessita nessita merged commit d50f61b into django:main Sep 16, 2024
34 checks passed
@LilyFirefly LilyFirefly deleted the patch-1 branch September 16, 2024 14:01
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.

3 participants