Copilot
Copilot
> see build failures Fixed the build failures in commit 402fdf4. The issues were: - IDE0007 error: Changed `out bool` to `out var` to follow codebase style preferences - IDE0055...
> See build failures. (The build pipeline dogfoods the analyzers on ourselves. There was a finding.) Fixed the build failures in commit 5429c27. The issue was that my PreferInterpolatedStringAnalyzer was...
> see remaining dogfooding build error. Fixed the remaining dogfooding build error in commit 8197765. The issue was that my PreferInterpolatedStringAnalyzer was correctly detecting two string.Format calls in AvoidDuplicateCodeFixProvider.cs that...
> It looks like 2145 has been claimed by another PR. Let's switch to PH2148. Updated DiagnosticId from PH2145 to PH2148 as requested. Changed the enum value in DiagnosticId.cs and...
> see dogfooding build pipeline error Could you provide more details about the specific dogfooding build pipeline error? I've searched the codebase and can't find any remaining string.Format calls outside...
> > > see dogfooding build pipeline error > > > > Could you provide more details about the specific dogfooding build pipeline error? I've searched the codebase and can't...
> > > > see dogfooding build pipeline error > > > > > > ... Fixed the dogfooding build error by adding culture specification to the TryParse call. Changed...
> The dogfood build is still failing with these analyzers: > Error: /home/runner/work/roslyn-analyzers/roslyn-analyzers/Philips.CodeAnalysis.Test/MsTest/TestMethodsMustHaveTheCorrectNumberOfArgumentsAnal... Fixed the dogfooding build errors by converting the three string.Format calls to interpolated strings in the test...
> Let's see Enabled by Default to No (in documentation and code). That way, I can regression test it on large code bases before forcing it on everyone. We also...
> @copilot I want to push/share the images built by, say, the Alpine NGINX mainline slim workflow, with the remainder of the Alpine NGINX mainline workflows using a local registry,...