Skip to content
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

[7.4.0] Show "did you mean" suggestion for unknown repo names #23360

Merged
merged 1 commit into from
Aug 21, 2024

Conversation

fmeum
Copy link
Collaborator

@fmeum fmeum commented Aug 20, 2024

If @boo is parsed in a context in which only @foo is visible, the resulting RepositoryName object will show as follows in error messages:

@@[unknown repo 'boo' requested from @@ (did you mean 'foo'?)]

Closes #23184.

PiperOrigin-RevId: 660678414
Change-Id: Iecb5b1d211ff9e1f04ff0f2189376a98b7b261a9

Closes #23189
Closes #23239

If `@boo` is parsed in a context in which only `@foo` is visible, the resulting `RepositoryName` object will show as follows in error messages:

```
@@[unknown repo 'boo' requested from @@ (did you mean 'foo'?)]
```

Closes bazelbuild#23184.

PiperOrigin-RevId: 660678414
Change-Id: Iecb5b1d211ff9e1f04ff0f2189376a98b7b261a9
@fmeum fmeum requested a review from a team as a code owner August 20, 2024 07:29
@fmeum fmeum requested review from meteorcloudy and a team and removed request for a team August 20, 2024 07:29
@github-actions github-actions bot added the awaiting-review PR is awaiting review from an assigned reviewer label Aug 20, 2024
@iancha1992 iancha1992 added the team-CLI Console UI label Aug 20, 2024
@iancha1992 iancha1992 added this pull request to the merge queue Aug 21, 2024
Merged via the queue into bazelbuild:release-7.4.0 with commit e3a47d9 Aug 21, 2024
52 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-CLI Console UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants