Skip to content
New issue

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

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

Already on GitHub? # to your account

make label parsing more resilient #7257

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

LeFrosch
Copy link
Collaborator

In some cases query sync might return some labels that cannot be validated later on. The actual fix would be to fix query sync to exclude these invalid labels; this just addresses the symptom.

@LeFrosch LeFrosch marked this pull request as ready for review January 27, 2025 12:38
@github-actions github-actions bot added product: CLion CLion plugin product: IntelliJ IntelliJ plugin product: GoLand GoLand plugin awaiting-review Awaiting review from Bazel team on PRs labels Jan 27, 2025
In some cases query sync might return some labels that cannot be
validated later on. The actual fix would be to fix query sync to exclude
these invalid labels; this just addresses the symptom.
@LeFrosch LeFrosch merged commit adb953a into bazelbuild:master Jan 31, 2025
5 checks passed
@LeFrosch LeFrosch deleted the label-validation branch January 31, 2025 09:32
@github-actions github-actions bot removed the awaiting-review Awaiting review from Bazel team on PRs label Jan 31, 2025
LeFrosch added a commit to LeFrosch/intellij-bazel that referenced this pull request Feb 3, 2025
In some cases query sync might return some labels that cannot be
validated later on. The actual fix would be to fix query sync to exclude
these invalid labels; this just addresses the symptom.
LeFrosch added a commit to LeFrosch/intellij-bazel that referenced this pull request Feb 3, 2025
In some cases query sync might return some labels that cannot be
validated later on. The actual fix would be to fix query sync to exclude
these invalid labels; this just addresses the symptom.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
product: CLion CLion plugin product: GoLand GoLand plugin product: IntelliJ IntelliJ plugin
Projects
Development

Successfully merging this pull request may close these issues.

4 participants