Skip to content

Conversation

@pmeric
Copy link
Collaborator

@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