bsorrentino / maven-confluence-plugin

Maven plugin that generates project's documentation directly to confluence allowing to keep in-sync project evolution with its documentation

Home Page:http://bsorrentino.github.io/maven-confluence-plugin/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Problem parsing % character as plain text

sebastianaguerre opened this issue · comments

Hi, I've been using this awesome plugin but I found that there are some issues when a % symbol is written onto the .md file as plain text

Let me show a couple of examples

... test coverage of at least 70%. (Measures to show this for each PR will need to be investigated)

Threw this error
image

And had to be changed to

... test coverage of at least 70 percent (Measures to show this for each PR will need to be investigated)

Another example (with the url changed - in bold the part that stays the same and has the problem)

For that, first you need to configure kubectl like described here: https://mypage.com/folder/anotherfolder/**guides_%26_docs**/yetanotherfolder

Threw this error

image

And had to be changed to be shown as a link to work

For that, first you need to configure kubectl like described here DOCUMENTATION

While this kind of error requires a workaround that is not that terrible to do, the main issue at least for me was to actually understand what was going on. I think it worth checking if there's something that can be done for this.

Hi @sebastianaguerre

Thanks to use such plugin and for your valuable feedbacks. I'll check asap

Hi @sebastianaguerre

I've find out the problem. during markdown parsing has been used String.format() and then % is marked as special char.

Currently you can add another % as workaround

test coverage of at least 70%%. (Measures to show this for each PR will need to be investigated)

Hi @sebastianaguerre

I've deployed version 7.9-SNAPSHOT with the fix. Please take a chance to test it so I can arrange a new production release

fix released in 7.9

Also ran into this exact issue. Nice seeing it already being solved.

There's is an issue with 7.9 not being available however:
https://search.maven.org/artifact/org.bsc.maven/confluence-reporting-maven-plugin
https://mvnrepository.com/artifact/org.bsc.maven/confluence-reporting-maven-plugin

Strangely enough I was able to fetch 7.9-SNAPSHOT and can confirm that the fix solves the issue with "%" in md files. (also figured out the "%%" workaround)

I would like however to use 7.9 and not 7.9-SNAPSHOT or 7.8+%% workaround so please lookinto the issue with 7.9 not being available