What does a red text-background mean in GitHub comparison view?

后端 未结 2 1148
我寻月下人不归
我寻月下人不归 2020-12-29 00:51

I created a pull request and was browsing through, the differences are shown with light red/green line-brackgrounds, but some text is bolded with a red text background...

相关标签:
2条回答
  • 2020-12-29 01:34

    This is issue makes code review process hard especially for the React(JSX) projects. Almost all the files with JSX showed the red lines.

    I fixed it by copy pasting

    var errorLine = document.getElementsByClassName("pl-ii");
    var i;
    for (i = 0; i < errorLine.length; i++) {
        errorLine[i].style.backgroundColor = "transparent";
        errorLine[i].style.color = "#24292e";
    }
    

    on my console

    0 讨论(0)
  • 2020-12-29 01:48

    The red background-color is definitely being caused by the error-highlighting feature of GitHub's text editor/viewer. You can find the same behaviour occurring to an intended block comment in another JSON file on GitHub:

    Screenshot of syntax error highlighting in GitHub

    As for your comment about some illegal characters not being highlighted: I also found that certain JSON errors aren't caught by GitHub's syntax processor. See this gist as an example:

    Example of uncaught errors in syntax highlighting

    In this case, the text outside of the outermost object isn't being highlighted. Whatever reason there is for this may be the same reason that errors aren't being highlighted for you.

    You can test it out for yourself by copy-pasting your code into a new Gist. Note that the ACE Editor has its own highlighting feature that can highlight code as you type, but its processing rules seem to be a bit different from that of GitHub's code viewer.

    0 讨论(0)
提交回复
热议问题