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

Support *.BUILD files #145

Merged
merged 2 commits into from
Jun 13, 2023
Merged

Support *.BUILD files #145

merged 2 commits into from
Jun 13, 2023

Conversation

pmeric
Copy link
Contributor

@pmeric pmeric commented May 26, 2023

bazel build files for external repos -- eg. used in conjunction with http_archive -- are often named <repo>.BUILD

bazel build files for external repos -- eg. used in conjunction with `http_archive` -- are often named <repo>.BUILD
Copy link
Contributor

@mco-gh mco-gh left a comment

Choose a reason for hiding this comment

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

thanks for the submission!

@mco-gh
Copy link
Contributor

mco-gh commented Jun 7, 2023

@willnorris Have you seen these lint errors before? Not sure what's triggering them but they seem unrelated to the PR I approved.

@willnorris
Copy link
Collaborator

oh, weird. No, I've not seen this before. The errors don't even really make sense for the code in addlicense.

I just checked, and our config for tailscale is mostly similar: and seems to be running like normal.

There are a couple of version differences with the actions/setup-go action and golangci-lint. addlicense isn't pinning to a specific version of the golangci-lint action, so maybe there's an issue with the latest version?

@mco-gh mco-gh merged commit b10ac1c into google:master Jun 13, 2023
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.

3 participants