allow documenter symbols within quoted blocks

[ZacLN]

closes #180

[timholy]

Thanks for fixing #180, but I have to gently complain that its test case was not included in this PR. There are absolutely times when it's a pain in the neck to write tests, but this was a case where all that was required was to copy/paste the reproducer and then check the result.

#166 calls for better test coverage, but this is the kind of change that won't affect coverage, because you were already running that line. Yet you know for certain that it was broken for specific inputs, so the coverage report should not be interpreted as a reason to stop worrying. If next year someone goes to rewrite some aspect of that block of code, now you've lost all guarantees that anything that worked formerly will continue to work.

For me it was an eye-opener to hear someone tell me to write the tests first, watch them fail, and then make the fix. I'm sure there are many sources to get this kind of advocacy from, but https://www.youtube.com/watch?v=HhwElTL-mdI (first ~15 minutes) is funny and well-put. I am not a "religious" person and so I never adopt anything unreservedly---for example, sometimes it's hard to isolate a sufficiently minimal test until you've started poking at the bug---but elements of TDD really do (once absorbed) make you not just safer, but actually faster.

[ZacLN]

Hi Tim, of course I can add a test. I'm not unenlightened to the benefits of of extensive set of tests (this package tests the validity of ~500,000 comparisons against the scheme parser's output) but am often pushed for time.

[davidanthoff]

One thing I would really love is a code coverage information that is not based on lines but on branches :) We have a lot of if clauses with quite complicated conditions, and in some sense the line based coverage information is really quite misleading for that kind of situation.

The text at https://marketplace.visualstudio.com/items?itemName=alexdima.vscode-lcov makes me think that the lcov file format somehow even supports that concept? I guess that is really something that would have to happen in Base, of course.