-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Error codes with single quote render poorly in Markdown #7195
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
Comments
Syntax highlighting on github is controlled by https://github.com/scala/vscode-scala-syntax, so maybe the highlighter could be fixed instead. |
Could the message just be T1 instead of T prime? Is the single tick significant? |
The problem is that T1 could be the name of a different type parameter, the type parameter is called T and the tick is used to differentiate different parameters with the same name. Alternatively we could use Unicode subscripts to append a number, or colors. |
Roger, so T prime means that it is T just not the same T - got it. Do you think we should create an issue for the syntax then and maybe leave this one here for reference for the time being or close this one? |
We could close this one for now, and reopen if we can't fix syntax highlighting. |
Created issue scala/vscode-scala-syntax#52 |
Possible fix scala/vscode-scala-syntax#59 |
This makes vscode-scala-syntax (and GitHub) highlight error messages correctly. Now it will print ```scala | where: T is a type in object B | T² is a type in object A ``` instead of ```scala | where: T is a type in object B | T' is a type in object A ```
This makes vscode-scala-syntax (and GitHub) highlight error messages correctly. Now it will print ```scala | where: T is a type in object B | T² is a type in object A ``` instead of ```scala | where: T is a type in object B | T' is a type in object A ```
Fix #7195: Use superscripts instead of `'` for duplicated types
minimized code
expectation
My hope would be that error messages like above could be modified to render nicely in Markdown since many people use UIs like GitHub.
The text was updated successfully, but these errors were encountered: