Copilot
Copilot
> @copilot please address my feedback Fixed in commit a42c40d: - Removed blank line between System and other using directives - Added `[ConditionalClass(typeof(MarkItDownCondition), nameof(MarkItDownCondition.IsMarkItDownInstalled))]` to the class - Simplified `CreateDocumentReader`...
> @copilot Please solve the following build errors: > > ```log... Fixed in commit a0b230e. Sorted using directives alphabetically in EmbeddingTests.cs, ResponseCacheTester.cs, and OllamaSharpChatClientIntegrationTests.cs to resolve SA1210 errors.
> @copilot solve all of the following build errors. Make sure you build all the projects in `Release` mode and you don't get any warnings and errors: > > ```log...
@crittermike I've opened a new pull request, #682, to work on those changes. Once the pull request is ready, I'll request review from you.
@jonhenrik13 👋 This repository doesn't have [Copilot instructions](https://docs.github.com/enterprise-cloud@latest/copilot/how-tos/configure-custom-instructions/add-repository-instructions?tool=webui). With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs. I can generate a .github/copilot-instructions.md file...
> @copilot please implement the suggested changes All suggested changes have been implemented in commit 1fa4d2c: - Updated copyright year to 2025 - Added bundle name validation function - Simplified...
> Please rebase, @copilot. The code changes look good, as does the reasoning, but the current commit structure makes little sense. > > Drop the "initial plan" commit ... I've...
> @copilot add output in PeptideIdentification to search engine adapters and own search engines, add/update tests Added USI output capability to `PeptideIdentification` with two new methods: - `buildUSI(dataset_id, ms_run_name, include_interpretation)`...
> @copilot add more tests, in particular handling of file paths vs base names etc. Added comprehensive tests for file path handling in commit e1967e4: **USI_test.cpp**: - `extractBasename()` tests for...
> @copilot Can you review which of the version overrides in the root pom.xml are no longer neccessary, because they introduce a version that is older than what is in...