Skip to content
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

See how default substitution rules can be combined #820

Closed
xcoulon opened this issue Feb 14, 2021 · 1 comment
Closed

See how default substitution rules can be combined #820

xcoulon opened this issue Feb 14, 2021 · 1 comment
Assignees
@xcoulon
Labels
area/parser type/refactor
Milestone
v0.7.0

Comments

@xcoulon
Copy link
Member

xcoulon commented Feb 14, 2021 •
edited
Loading

For now, each substitution rule is applied separately, which means that a block is parsed multiple times. This works fine, but it's could be faster if some rules could be combined. At least for the default cases, i.e., when no subs attribute is specified.
@xcoulon xcoulon added this to the v0.6.0 milestone Feb 14, 2021
@xcoulon xcoulon self-assigned this Feb 14, 2021
@xcoulon xcoulon changed the title See how substitution rules can be combined See how default substitution rules can be combined Feb 14, 2021
@xcoulon xcoulon modified the milestones: v0.6.0, backlog Mar 21, 2021
@xcoulon
Copy link
Member Author

xcoulon commented Mar 27, 2022

closing, done in #855

@xcoulon xcoulon closed this as completed Mar 27, 2022
@xcoulon xcoulon modified the milestones: backlog, v0.7.0 Mar 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xcoulon xcoulon

Labels
area/parser type/refactor
Projects
None yet
Milestone
v0.7.0
Development

No branches or pull requests
1 participant
@xcoulon