Skip to content

Conversation

@ianjosephwilson
Copy link
Contributor

  • Remove anonymous fragments from the parser for now: ie. <></>.

@pauleveritt
Copy link
Contributor

That's cool that you detect it and raise NotImplemented. You removed all mention from the README, I believe. Should we have a quick callout saying "some of you might be looking for....but not needed"?

@ianjosephwilson
Copy link
Contributor Author

Well it doesn't really detect per se I just wasn't sure what the best way to handle the type checking without drowning in trying to splice that class out of the union type. We are still using implicit fragments to wrap each t-string so we need some of that code in there. I think we'll need to work on the messaging around that and maybe put it in the fragment issue.

@davepeck davepeck merged commit 562b91c into t-strings:main Dec 19, 2025
1 check passed
@ianjosephwilson ianjosephwilson deleted the ian/remove_anonymous_fragment branch January 1, 2026 06: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