Skip to content

Sync lockfile #26339

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Sync lockfile #26339

wants to merge 1 commit into from

Conversation

keith
Copy link
Member

@keith keith commented Jun 18, 2025

Done with

bazel mod graph --lockfile_mode=update

@github-actions github-actions bot added the awaiting-review PR is awaiting review from an assigned reviewer label Jun 18, 2025
@iancha1992 iancha1992 added the team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file. label Jun 19, 2025
Done with

```
bazel mod graph --lockfile_mode=update
```
@keith keith force-pushed the ks/sync-lockfile branch from eed6a04 to 2b2cc2c Compare June 20, 2025 17:54
@keith keith requested review from meteorcloudy and Wyverald June 20, 2025 18:00
@Wyverald
Copy link
Member

I wonder how this happened... since #26354 contains this, let's just merge that one instead.

@Wyverald Wyverald closed this Jun 23, 2025
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Jun 23, 2025
@keith keith deleted the ks/sync-lockfile branch June 23, 2025 19:53
@fmeum
Copy link
Collaborator

fmeum commented Jun 23, 2025

My bet is on a merge conflict resolution which added both sides of the merge. --lockfile_mode=error doesn't error on extra entries (and I think that's a good thing).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants