fix: enable jinja templater for previously ignored test fixtures#2220
Draft
fix: enable jinja templater for previously ignored test fixtures#2220
Conversation
Update rule test fixtures that were ignored due to missing jinja
templater configuration. For each test that uses jinja syntax
({{ }}, {% %}), add `templater: jinja` to the configs section,
allowing these tests to run properly.
This enables ~80 tests across 20 fixture files that were previously
skipped with "jinja is not set" or "jinja is not supported" messages.
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Benchmark for dc97039Click to view benchmark
|
|
Looks like there are a few issues preventing this PR from being merged!
If you'd like me to help, just leave a comment, like Feel free to include any additional details that might help me get this PR into a better state. You can manage your notification settings |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
{{ }},{% %}), addtemplater: jinjato configsTest plan
cargo testto verify all updated fixtures pass🤖 Generated with Claude Code