Youssef Victor

Results 913 comments of Youssef Victor

Closing stale PR, though I *do* think the changes proposed here make sense.

I just tested, seems not handled. ```csharp using System.Diagnostics; namespace TestProject87; [TestClass] public sealed class Test1 { [TestMethod] public void TestMethod1() { } [TestMethod] public void TestMethod2() { // --results-directory...

I am never a big fan of `Single` API because when it throws it's hard to debug, you don't know what was there. _Originally posted by @Evangelink in https://github.com/microsoft/testfx/pull/5717#discussion_r2140885626_

This is curious. If RenderSize got updated, then SizeChanged should be enqueued. That then means it's enqueued but there is no new frame so it's not fired. After we set...

Thnking about this again, RequestAdditionalFrame shouldn't be needed. RenderSize is supposed to change during Arrange. Then, after arrange is completed, we should be hitting this path: https://github.com/unoplatform/uno/blob/5fd8ce0b9eb632829840cdd491af2275b4b38909/src/Uno.UI/UI/Xaml/UIElement.cs#L860

You can do it this way: ```csharp TestState.AnalyzerConfigFiles.Add(("/.globalconfig", """ is_global = true build_property.SomeProperty = value """)); ```

@joem-msft Can I get a review on this PR please?

@mesganmsft Thanks for reviewing! Is this good to merge now?

@mizrael Your point is totally valid. CI shouldn't list the test projects and run on them one by one. A couple of reasons: 1. When a new test project is...

@cxw42 It will be usefuly for the C# compiler. (Note: I'm not a Microsoft employee, but a regular contributor to Roslyn) In short, the C# compiler currently uses .editorconfig files...