Cédric Luthi

Results 226 comments of Cédric Luthi

What bothers me is that there's no difference between _pass_ and _warn_ at the HTTP status code level as both _MUST return HTTP status in the 2xx-3xx range_. Imagine you're...

I just tried again with jazzy 0.7.3 and the *Empty Type Definitions entry in the sidebar with a broken link* issue seems solved. All the other issues are still not...

Thanks for your suggestion. I will consider adding this feature in a future version of XCDYouTubeKit.

#1877 was reverted in #1996 so that fixes the _log file name timestamp_ part of the issue. Being able to run tests on multiple frameworks and collect the results is...

I just stumbled on this too and was somewhat disappointed that the default assertion message does not include the HTTP body. Like Bart described, displaying the body (which contains an...

I regularly stumble on this when writing tests and would love to see @dennisdoomen's proposal to add `[System.Diagnostics.CodeAnalysis.NotNull]` to `Should` implemented in FluentAssertions. I also agree that _in the end...

Thank you, it works fine for rendering the state history. I don't actually need to add a widget, I just want to remove the useless quotes on the `Result`. I...

My implementation of the same feature (#1317) was a little bit more complete with strongly typed `HttpRequestOptionsKey` and with updates to the README. Maybe you should also consider merging it.

> What about extending the abstract builder and adding a member for setting the ILogger implementation, like WithLogger(ILogger)? Just after filing this issue I realized that this may be a...

> However, it requires a more effective approach to log container runtime information. Do you mean that you don't like logging at the very beginning of `TestcontainersClient.RunAsync` because it could...