-
Notifications
You must be signed in to change notification settings - Fork 14
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
The escape char is not showing up on the UI #91
Comments
The above code is appearing correctly on the lgdev site as (\\s+). But is showing up incorrectly as (\s+) on the QA site That line of code is new btw (added for dev mode) in branch |
I'm still looking into why this could be different on the QA environment. By the time the MetricsIT.java code gets loaded into html in the browser it already has one of the backslashes missing so it's not javascript that is doing this. And both environments have the same version of asciidoctor. I'm still lost as to why it is different but I'll keep looking. |
temporary updated the guide by |
pending a fix for OpenLiberty/openliberty.io#1975 |
Looks like the issue is in the minifier (https://github.com/octopress/minify-html) but unfortunately it doesn't seem like it's being actively maintained anymore (though we can certainly try and open an issue). In the meantime, I suggest changing the example/test code to avoid this scenario. |
The test class has the following line of code that uses escape char:
1 <= Integer.parseInt(metric.split("\\s+")[metric.split("\\s+").length - 1]));
It shows up on the ol.io UI as:
1 <= Integer.parseInt(metric.split("\s+")[metric.split("\s+").length - 1]));
On Linux and Mac, the later causes a code compilation error:
The text was updated successfully, but these errors were encountered: