docs: add documentation of updateThreadInfo setting#197
docs: add documentation of updateThreadInfo setting#197jreineckearm merged 1 commit intoeclipse-cdt-cloud:mainfrom
Conversation
cwalther
left a comment
There was a problem hiding this comment.
Thanks for the starting point!
a7650d0 to
92e22f3
Compare
There was a problem hiding this comment.
Looks good to me now, thanks. Any comments from @jreineckearm before we merge?
Nitpick (not a blocker): Now that the list is outside of the table, you could use Markdown syntax for it again instead of HTML.
By the way, my comments were meant as discussion inputs, not as orders, but if you agree with them that’s fine too.
jreineckearm
left a comment
There was a problem hiding this comment.
Thanks for the docs updates! LGTM in general.
I agree with @cwalther that we should avoid HTML in markdown files if possible. In this case, - for bullet items and backticks instead of <code>. But not necessarily a blocker to get this done.
Good to merge unless you'd like to make any last changes, @andy9a9 .
|
Fine with me, but if I have to change it, no problem, but let me know. |
|
OK, will merge then. |
Document new setting from eclipse-cdt-cloud/cdt-gdb-adapter#482