Skip to content

Enable buildifier on CI#529

Open
keith wants to merge 7 commits into
bazelbuild:mainfrom
keith:ks/enable-buildifier-on-ci
Open

Enable buildifier on CI#529
keith wants to merge 7 commits into
bazelbuild:mainfrom
keith:ks/enable-buildifier-on-ci

Conversation

@keith

@keith keith commented Jun 26, 2026

Copy link
Copy Markdown
Member

No description provided.

@keith keith force-pushed the ks/enable-buildifier-on-ci branch from ddc97a6 to 42907f1 Compare June 26, 2026 16:40
@keith keith marked this pull request as ready for review June 26, 2026 17:21
Comment thread mobile_install/utils.bzl
)

def _get_extension_registry_class_jar(target):
def _get_extension_registry_class_jar(target): # @unused

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume this is used in google? otherwise we can toss this here instead?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

im trying to toss all the unused stuff in #530 so we can discuss that there

Comment thread mobile_install/apks.bzl Outdated
@keith

keith commented Jun 26, 2026

Copy link
Copy Markdown
Member Author

it's probably possible that we need to keep some of the load() statements for google specific targets that are filtered out on export? lmk if so

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant