ci: remove unused dependency from BUILD targets #37843
Conversation
agrawroh
requested review from
jmarantz,
mattklein123 and
alyssawilk
as code owners
agrawroh
force-pushed
the
rm_unused_dep
branch
from
e1c3ba5 to
5421fc7
Compare
Signed-off-by: Rohit Agrawal <[email protected]>
agrawroh
force-pushed
the
rm_unused_dep
branch
from
5421fc7 to
619369e
Compare
|
/retest |
|
I manually retriggered CodeQL-Build and it still failed the same way. Doesn't look like it's directly related to your PR, maybe try syncing up to head? |
@ravenblackx I tried syncing with upstream but it's still failing with the same error :| |
|
/retest |
|
/retest Not quite the same error, but another "absl missing its own internal dependencies" error. This looks similar, I suspect something is off in the build system (bad cache or incompatible versions, kinda thing). I don't know how to unblock though. |
|
Looks like we can just ignore this if we're confident this PR is not causing it, and I'm pretty confident of that. |
## Description This PR removes an unused dependency from multiple BUILD targets and header files. --- **Commit Message**: remove unused dependency from BUILD targets **Additional Description:** This PR removes unused dependency from multiple header and BUILD files. **Risk Level:** Very Low Testing: N/A **Docs Changes:** N/A **Release Notes:** N/A Signed-off-by: Rohit Agrawal <[email protected]>
Description
This PR removes an unused dependency from multiple BUILD targets and header files.
Commit Message: remove unused dependency from BUILD targets
Additional Description: This PR removes unused dependency from multiple header and BUILD files.
Risk Level: Very Low
Testing: N/A
Docs Changes: N/A
Release Notes: N/A