Skip to content

Remove the stray uv.lock from connectrpc-grpcreflect - #354

Merged
stefanvanburen merged 1 commit into
mainfrom
svanburen/remove-grpcreflect-lock
Sep 25, 2026
Merged

stefanvanburen merged 1 commit into
mainfrom
svanburen/remove-grpcreflect-lock

Conversation

@stefanvanburen

@stefanvanburen stefanvanburen commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Seems like we haven't been getting Renovate updates for our security alerts, and one theory that Claude had was this stray introduced uv.lock file, which isn't needed since it's using the workspace-level uv.lock.

Merging this before Renovate's next scheduled run (early Monday, September 28) tests whether this file is what stopped the weekly lock file maintenance PRs. If a "Lock file maintenance" PR appears, it should also clear the open Dependabot alerts for anyio and h2, which are only reachable through uv.lock. If none appears, the hosted Renovate logs at https://developer.mend.io/github/connectrpc/connect-py are the next place to look.

`connectrpc-grpcreflect` is a workspace member, so uv resolves it
from the root `uv.lock` and never reads or updates its own lock file;
this one has been untouched since #292 added it.

Renovate still treats it as that package's lock file. Its weekly lock
file maintenance PRs stopped after #290, the last one before #292
landed, which leaves transitive dependencies such as `anyio` and `h2`
on versions with open Dependabot alerts. Whether this file is the cause
is unconfirmed; the next scheduled Renovate run will tell.

Signed-off-by: Stefan VanBuren <stefan@vanburen.xyz>

@anuraaga anuraaga left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙏

@stefanvanburen
stefanvanburen merged commit bb5a723 into main Sep 25, 2026
24 checks passed
@stefanvanburen
stefanvanburen deleted the svanburen/remove-grpcreflect-lock branch September 25, 2026 13:37
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