Dotnet migration - #50
Merged
Merged
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Without this, local Windows build artifacts (obj/project.assets.json) were copied into the container, overwriting the clean restore and causing a failure due to a Windows-only NuGet fallback folder path. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Add EnableLlm bool to BottomlyOptions (defaults false) - Read EnableLlm from configuration in Program.cs - Wrap all Ollama/LLM DI registrations in EnableLlm guard - Exclude ConversationMessageHandler from auto-registration when disabled - Set EnableLlm: false in appsettings.json Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
ServiceDefaults pulls in Microsoft.AspNetCore.App via OpenTelemetry and health check dependencies, requiring the ASP.NET Core runtime. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Remove -f run.dockerfile flag, use default Dockerfile - Replace bottomly_mongo_conn_str with ConnectionStrings__mongodb to match Aspire connection string conventions Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ig keys - dotnet workload install aspire requires --include-previews to install the preview Aspire workload manifest in CI - NuGet.config source keys were set to URLs instead of friendly names, which can cause packageSourceMapping to fail to match in CI Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The dotnet9 feed does not carry .NET 10 Aspire preview packages. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Aspire.Hosting 13.3.0-preview.1.26156.8 is published to the dotnet9 feed, not dotnet10. Also reverts AppHost package versions back to the correct 13.3.0-preview.1.26156.8 after failed downgrade attempts. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The dotnet CLI searches for NuGet.config in the current directory and parent directories, but does NOT search .nuget/ subdirectories. The file in .nuget/ was only working locally because Visual Studio has its own fallback feed configs. On CI (GitHub Actions), dotnet restore only saw nuget.org and could not find the preview Aspire packages from the dotnet9 Azure DevOps feed. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.