Mathias Stearn

Can `doctest::String` be less required in the public API?

3

There is already `DOCTEST_CONFIG_DOUBLE_STRINGIFY` so that `toString()` implementations can use a normal string type. It would be nice if the rest of the API could be made to not depend...

Is separate header and cpp a supported build method?

1

When vendoring doctest, it seems like it would be better (for several reasons) to vendor and build the separate `doctest_fwd.h` and `doctest.cpp` files rather than using the amalgamated megaheader with...

Can `detail::throwException()` and `detail::TestFailureException` be made public?

1

This would solve 2 problems. The more important one is that when shimming in doctest under our existing testing API I need to support the equivalent of `REQUIRE_EQ(a, b)

The query flags like "list-foo" and "help" don't support the `--list-foo=true` syntax which is what `context.setOption("list-foo", true)` desugars to. They probably should be handled exactly like bool flags. Also, or...

**What this PR does / why we need it**: Exactly what the title says. See linked issue for why. **Which issue(s) this PR fixes**: fixes #476. **Special notes for your...

**Checklist** - [x] The issue is about this extension and NOT about a fork. - [x] Checked the [ALL the SUPPORT](https://github.com/matepek/vscode-catch2-test-adapter/blob/master/documents/support.md) document. - [x] The latest version of the extension...

Support the colorMoved setting on git diff

1

It may be worth using it by default. But at the very least it would be nice it would honor the `diff.colorMoved` gitconfig setting when viewing diffs in tig.

Please support the new ${workspaceFolder} replacement for ${workspaceRoot}

1

See https://code.visualstudio.com/docs/reference/variables-reference#_why-isnt-workspaceroot-documented where it explicitly says that > The variable `${workspaceRoot}` was deprecated in favor of `${workspaceFolder}` (The cookie banner covers that text if you haven't accepted it yet) You...

Add --no-follow to ignore symlinks when setting watches

3

- Watchexec's version: release: 2.3.2 features: default,pid1 - The OS you're using: Linux `watchexec` seems to favor paths though symlinks if it encounters them first, even if the symlink path...

There are a few reasons I think this is worth doing. The most pressing is that it provides compatibility with tools such as `protoc` which can generate mocks using `MOCK_METHOD()`....