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.

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

feat(parser): support custom substitutions on delimited blocks #759

Conversation

xcoulon
Copy link
Member

@xcoulon xcoulon commented Sep 30, 2020

adding missing tests for custom subs on verse blocks at the
parser level, and for all subs at the rendering level

also, fix missing "InlineMacros" rule in the "VerseMacro" subs.

also, reorgnize tests so they all have the same structure,
in the parser and renderer pkgs.

Signed-off-by: Xavier Coulon [email protected]

adding missing tests for custom subs on verse blocks at the
parser level, and for all subs at the rendering level

also, fix missing "InlineMacros" rule in the "VerseMacro" subs.

also, reorgnize tests so they all have the same structure,
in the parser and renderer pkgs.

Signed-off-by: Xavier Coulon <[email protected]>
@codecov
Copy link

codecov bot commented Sep 30, 2020

Codecov Report

Merging #759 into master will increase coverage by 0.59%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #759      +/-   ##
==========================================
+ Coverage   81.25%   81.85%   +0.59%     
==========================================
  Files          76       76              
  Lines        5014     4992      -22     
==========================================
+ Hits         4074     4086      +12     
+ Misses        606      572      -34     
  Partials      334      334              

@xcoulon xcoulon merged commit 667d4ce into bytesparadise:master Sep 30, 2020
@xcoulon xcoulon deleted the Issue558_custom_block_substitutions_tests branch September 30, 2020 21:28
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.

1 participant