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

fix(parser): support commas in link text attribute #389

Merged

Conversation

xcoulon
Copy link
Member

@xcoulon xcoulon commented Aug 31, 2019

supoort text with comma and without being surrounded
by double quotes if no other attribute is defined,
or require the (positional) text attribute to be
surrounded with double quotes if it contains commas
and there are other attributes in the link macro.

fixes #388

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

supoort text with comma and without being surrounded
by double quotes if no other attribute is defined,
or require the (positional) text attribute to be
surrounded with double quotes if it contains commas
and there are other attributes in the link macro.

fixes bytesparadise#388

Signed-off-by: Xavier Coulon <[email protected]>
@xcoulon xcoulon merged commit 23feea9 into bytesparadise:master Aug 31, 2019
@codecov
Copy link

codecov bot commented Aug 31, 2019

Codecov Report

Merging #389 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #389   +/-   ##
=======================================
  Coverage   81.47%   81.47%           
=======================================
  Files          48       48           
  Lines        4297     4297           
=======================================
  Hits         3501     3501           
  Misses        659      659           
  Partials      137      137

@xcoulon xcoulon deleted the Issue388_link_description_with_comma branch August 31, 2019 14:52
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.

Comma inside link's description clear the rest of characters
1 participant