Skip to content

Support custom simple block tags #128

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 17 commits into
base: main
Choose a base branch
from
Draft

Support custom simple block tags #128

wants to merge 17 commits into from
+721 −108

Conversation

LilyFirefly
Copy link
Owner

No description provided.

@LilyFirefly LilyFirefly self-assigned this Sep 22, 2025
@LilyFirefly LilyFirefly added enhancement New feature or request refactor labels Sep 22, 2025
@codecov Codecov
Copy link

codecov bot commented Sep 22, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 95.16539% with 19 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/parse.rs 92.40% 0 Missing and 12 partials ⚠️
src/render/tags.rs 93.45% 0 Missing and 7 partials ⚠️

📢 Thoughts on this report? Let us know!

@LilyFirefly LilyFirefly force-pushed the custom-simple-block-tags branch 2 times, most recently from c3b0a11 to e4e40be Compare September 22, 2025 13:50
@LilyFirefly LilyFirefly force-pushed the custom-simple-block-tags branch 4 times, most recently from fd37db5 to 4045595 Compare October 2, 2025 20:22
@LilyFirefly
Use Cow<'static, str> instead of &'static str for end tag names
d361a8e 
This will allow using dynamic end tag names (i.e. for custom tags)
without needing to allocate a `String` for a known-at-compile-time end
tag name.
@LilyFirefly
Move custom tag tests to only cover simple custom tags
3407f9b
@LilyFirefly
Extract context wrangling into helper functions
f906ccd 
This will allow re-use in `SimpleBlockTag`.
@LilyFirefly
Rename TagContext::SimpleTag to TagContext::Simple
f846af9
@LilyFirefly
Make call_tag a standalone function
efa5460 
This will allow it to be used in `SimpleBlockTag::render` in addition
to `SimpleTag::render`.
@LilyFirefly
Split handling of defaults for custom tags into helper functions
9891ea6
@LilyFirefly
Implement custom simple block tags
41c4d77
@LilyFirefly
Add error handling for missing context and/or content
b12279d
@LilyFirefly
Test simple_block_tag with context and end_name
28ff376
@LilyFirefly
Test missing/only end tag for simple_block_tag
69f5aad
@LilyFirefly
Test invalid argument handling within custom simple block
b53d2a2
@LilyFirefly
Refactor handling of missing variables in custom tags
3cf6617 
A missing variable should be treated as an empty string and passed
through to the custom tag.
@LilyFirefly
Extract build_kwargs helper function for custom tags
0440ad3
@LilyFirefly
Test PartialEq for SimpleBlockTag
a3fff57
@LilyFirefly
Test syntax error within custom block tag
4200cb7
@LilyFirefly LilyFirefly force-pushed the custom-simple-block-tags branch from 24ae907 to 4200cb7 Compare October 5, 2025 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@LilyFirefly LilyFirefly

Labels
enhancement New feature or request refactor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@LilyFirefly