Refresh Couchbase .NET dependencies and docs#144
Open
dex-the-ai wants to merge 2 commits intomainfrom
Open
Conversation
Author
|
Owner ping for this maintenance PR: @azaddhirajkumar Requesting review from the manifest-listed owner(s) where GitHub does not already show a pending review request. |
There was a problem hiding this comment.
Code Review
This pull request updates the project to .NET 8 and upgrades several dependencies, including the Couchbase SDK, BCrypt, xUnit, and Newtonsoft.Json. Additionally, the README has been updated to document environment variable overrides for database configuration. A review comment suggests clarifying that all three environment variables—DB_CONN_STR, DB_USERNAME, and DB_PASSWORD—must be set for the override to take effect, preventing potential configuration issues.
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
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.
Summary
3.9.xand bumpBCrypt.Net-Next,xunit, andNewtonsoft.Json.NET 8+baselineDB_CONN_STR/DB_USERNAME/DB_PASSWORDoverride path so local users do not need to edit tracked config with secretsVerification
dotnet test Org.Quickstart.sln --no-build --verbosity normaldotnet list Org.Quickstart.sln package --vulnerablehttp://127.0.0.1:8098Evidence
tutorial-maintenance/runs/aspnet-quickstart/2026-05-05T0145Z-subagent/verification.mdNotes
.NET 8install because the host default SDK was newer than the repo target.