Skip to content

[AOSP-pick] Exclude no-ide targets from query results #7441

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

Merged
merged 1 commit into from
Mar 7, 2025

Conversation

sellophane
Copy link
Collaborator

Cherry pick AOSP commit a4267596e72fdfc3b22d6bbdc93ca15b181ac80a.

  1. Filter out no-ide targets in the query phase.
  2. Re-enable SyncedInBazelProjectTest.javaAndDeps test, which proves
    that no-ide is targets are not in the query results.

Bug: n/a
Test: SyncedInBazelProjectTest.javaAndDeps
Change-Id: I7a0405a70415cb1e24ebe96b84e36643958fe14a

AOSP: a4267596e72fdfc3b22d6bbdc93ca15b181ac80a

@sellophane sellophane requested a review from mai93 as a code owner March 7, 2025 12:43
@sellophane sellophane requested a review from LeFrosch March 7, 2025 12:43
@github-actions github-actions bot added the awaiting-review Awaiting review from Bazel team on PRs label Mar 7, 2025
1. Filter out `no-ide` targets in the query phase.
2. Re-enable `SyncedInBazelProjectTest.javaAndDeps` test, which proves
   that `no-ide` is targets are not in the query results.

Bug: n/a
Test: SyncedInBazelProjectTest.javaAndDeps
Change-Id: I7a0405a70415cb1e24ebe96b84e36643958fe14a

AOSP: a4267596e72fdfc3b22d6bbdc93ca15b181ac80a
@LeFrosch LeFrosch force-pushed the AOSP/a4267596e72fdfc3b22d6bbdc93ca15b181ac80a branch from 9018215 to b13bb8a Compare March 7, 2025 13:21
@sellophane sellophane merged commit 840b622 into master Mar 7, 2025
6 of 7 checks passed
@github-project-automation github-project-automation bot moved this from Untriaged to Done in Bazel IntelliJ Plugin Mar 7, 2025
@github-actions github-actions bot removed the awaiting-review Awaiting review from Bazel team on PRs label Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants