Skip to content

fix(deps): regenerate sdk_compliance_adapter lock file - #310

Merged
turnipdabeets merged 1 commit into
mainfrom
fix/adapter-lockfile
Aug 28, 2026
Merged

fix(deps): regenerate sdk_compliance_adapter lock file#310
turnipdabeets merged 1 commit into
mainfrom
fix/adapter-lockfile

Conversation

@turnipdabeets

Copy link
Copy Markdown
Contributor

#309 bumped the central package versions but left sdk_compliance_adapter/packages.lock.json pinned to the old ones (Microsoft.Extensions.Configuration [9.0.1, ) vs 10.0.11 in Directory.Packages.props), so the adapter's Docker build fails dotnet restore --locked-mode and the SDK compliance job is red on main.

The adapter is not in PostHog.sln, so the solution-wide --force-evaluate that fixed the other ten lock files missed it. That job also isn't a required check — it only emits a ::warning:: — so #309 merged with it failing.

Regenerated with dotnet restore sdk_compliance_adapter/SdkComplianceAdapter.csproj --force-evaluate. Verified locked-mode restore and build pass on top of main.

#309 bumped the central package versions but left this lock file at the old
ones, so the adapter's Docker build fails `dotnet restore --locked-mode`. The
adapter is not in PostHog.sln, so the solution-wide regeneration missed it, and
the SDK compliance job is not a required check, so #309 merged red.
@turnipdabeets
turnipdabeets requested a review from a team as a code owner August 28, 2026 18:47
@turnipdabeets
turnipdabeets enabled auto-merge (squash) August 28, 2026 18:48
@greptile-apps

greptile-apps Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "fix(deps): regenerate sdk_compliance_ada..." | Re-trigger Greptile

@github-actions

Copy link
Copy Markdown
Contributor

posthog-dotnet Compliance Report

Date: 2026-08-28 18:49:36 UTC
Duration: 4337ms

✅ All Tests Passed!

17/17 tests passed


Feature_Flags Tests

17/17 tests passed

View Details
Test Status Duration
Request Payload.Request With Person Properties Device Id 239ms
Request Payload.Flags Request Uses V2 Query Param 127ms
Request Payload.Flags Request Hits Flags Path Not Decide 112ms
Request Payload.Flags Request Omits Authorization Header 111ms
Request Payload.Token In Flags Body Matches Init 111ms
Request Payload.Groups Round Trip 114ms
Request Payload.Groups Default To Empty Object 111ms
Request Payload.Disable Geoip False Propagates As Geoip Disable False 112ms
Request Payload.Disable Geoip Omitted Defaults To False 112ms
Request Payload.Flag Keys To Evaluate Contains Only Requested Key 111ms
Request Lifecycle.No Flags Request On Init Alone 5ms
Request Lifecycle.No Flags Request On Normal Capture 116ms
Request Lifecycle.Two Flag Calls Produce Two Remote Requests 215ms
Request Lifecycle.Mock Response Value Is Returned To Caller 116ms
Retry Behavior.Retries Flags On 502 1117ms
Retry Behavior.Retries Flags On 504 1117ms
Side Effect Events.Get Feature Flag Captures Feature Flag Called Event 215ms

@turnipdabeets
turnipdabeets merged commit 71841e1 into main Aug 28, 2026
20 checks passed
@turnipdabeets
turnipdabeets deleted the fix/adapter-lockfile branch August 28, 2026 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants