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 #755

Merged

Conversation

xcoulon
Copy link
Member

@xcoulon xcoulon commented Sep 26, 2020

applying the same logic as with paragraphs to apply default and
custom substitutions on delimited blocks. This means that
delimited blocks are processed in multiple steps, but each step
has a much simpler grammar.

Also, quite a lot of refactoring and code reorganization for the
delimited block tests in pkg/parser.

See #749

Fixes #558
Signed-off-by: Xavier Coulon [email protected]

@xcoulon xcoulon marked this pull request as draft September 26, 2020 19:32
@codecov
Copy link

codecov bot commented Sep 26, 2020

Codecov Report

Merging #755 into master will decrease coverage by 2.27%.
The diff coverage is 73.97%.

@@            Coverage Diff             @@
##           master     #755      +/-   ##
==========================================
- Coverage   83.52%   81.25%   -2.28%     
==========================================
  Files          77       76       -1     
  Lines        5603     5014     -589     
==========================================
- Hits         4680     4074     -606     
- Misses        586      606      +20     
+ Partials      337      334       -3     

@xcoulon xcoulon force-pushed the Issue558_custom_block_substitutions branch 2 times, most recently from aa11dcf to 6c42006 Compare September 28, 2020 20:15
@xcoulon xcoulon marked this pull request as ready for review September 28, 2020 20:16
@xcoulon xcoulon force-pushed the Issue558_custom_block_substitutions branch from 6c42006 to adc13f4 Compare September 28, 2020 20:19
applying the same logic as with paragraphs to apply default and
custom substitutions on delimited blocks. This means that
delimited blocks are processed in multiple steps, but each step
has a much simpler grammar.

Also, quite a lot of refactoring and code reorganization for the
delimited block tests in `pkg/parser`.

See bytesparadise#749

Fixes bytesparadise#558

Signed-off-by: Xavier Coulon <[email protected]>
@xcoulon xcoulon force-pushed the Issue558_custom_block_substitutions branch from adc13f4 to 3cf0426 Compare September 28, 2020 20:20
@xcoulon xcoulon merged commit 2179d24 into bytesparadise:master Sep 28, 2020
@xcoulon xcoulon deleted the Issue558_custom_block_substitutions branch September 28, 2020 20:27
@xcoulon
Copy link
Member Author

xcoulon commented Sep 28, 2020

finally! 😅

xcoulon added a commit to xcoulon/libasciidoc that referenced this pull request Sep 28, 2020
issue was fixed by bytesparadise#755, verifying it here

Fixes bytesparadise#748

Signed-off-by: Xavier Coulon <[email protected]>
xcoulon added a commit that referenced this pull request Sep 28, 2020
issue was fixed by #755, verifying it here

Fixes #748

Signed-off-by: Xavier Coulon <[email protected]>
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.

Custom substitutions on blocks
1 participant