basilisp-lang / basilisp / 10743948749

99%

push

github

nREPL server should not send color ANSI esc seqs to the clients (#1040)

Hi,

could you please review patch to prevent the nREPL server sending color
ANSI esc seqs to the clients. It fixes #1039.

I modified the path from the nREPL server down to the Pygments call to
accept a `disable_color` parameter, which stops color formatting.

I've also updated the tests to reflect this change. The
`src/basilisp/lang/source.py` test is currently placed under
`tests/basilisp/lang`, though I noticed the other tests in the
`src/basilisp/` subdirectories don't follow this structure -- not sure
why that is. I'm happy to relocate it to `tests/basilisp` if needed.

Thanks

---------

Co-authored-by: ikappaki <ikappaki@users.noreply.github.com>

1870 of 1877 branches covered (99.63%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 2 files covered. (100.0%)

8547 of 8656 relevant lines covered (98.74%)

0.99 hits per line