Skip to content

fix(fast-html): refactor naming for binding syntax and add a test for tag collisions#7335

Merged
janechu merged 8 commits intomainfrom
users/janechu/add-a-binding-configuration
Mar 24, 2026
Merged

fix(fast-html): refactor naming for binding syntax and add a test for tag collisions#7335
janechu merged 8 commits intomainfrom
users/janechu/add-a-binding-configuration

Conversation

@janechu
Copy link
Copy Markdown
Collaborator

@janechu janechu commented Mar 21, 2026

Pull Request

📖 Description

A syntax interface has been added to help keep syntax logic in a single place. This is a small refactor in case of collisions with other templating syntax require a feature where users can modify the syntax themselves. Additionally, a small update was made to the logic around f-when and f-repeat to avoid collisions in case the user adds their own f-* tags.

✅ Checklist

General

  • I have included a change request file using $ npm run change
  • I have added tests for my changes.
  • I have tested my changes.
  • I have updated the project documentation to reflect my changes.
  • I have read the CONTRIBUTING documentation and followed the standards for this project.

@janechu janechu force-pushed the users/janechu/add-a-binding-configuration branch from 7f74663 to 04f37ad Compare March 21, 2026 02:31
@janechu janechu force-pushed the users/janechu/add-a-binding-configuration branch from 17e83d9 to 3e03b7f Compare March 24, 2026 16:36
@janechu janechu requested a review from radium-v March 24, 2026 16:38
@janechu janechu force-pushed the users/janechu/add-a-binding-configuration branch from 3e03b7f to 66dc28a Compare March 24, 2026 17:41
radium-v
radium-v previously approved these changes Mar 24, 2026
@janechu janechu merged commit ca63fe8 into main Mar 24, 2026
12 checks passed
@janechu janechu deleted the users/janechu/add-a-binding-configuration branch March 24, 2026 22:29
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