Skip to content

Conversation

@stefanb
Copy link
Member

@stefanb stefanb commented Sep 4, 2025

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

pin go to [email protected] after failing in

upstream ref

@github-actions github-actions bot added the go Go use is a significant feature of the PR or issue label Sep 4, 2025
@stefanb stefanb added the CI-no-bottles Merge without publishing bottles label Sep 4, 2025
@stefanb stefanb changed the title ibazel: test, rebuild ibazel: pin go to [email protected] Sep 4, 2025
@branchv
Copy link
Member

branchv commented Sep 4, 2025

I think we just need to bump the rules_go to the latest (0.57.0)

bazel_dep(name = "rules_go", version = "0.55.1")

Signed-off-by: Rui Chen <[email protected]>
@chenrui333
Copy link
Member

I think we just need to bump the rules_go to the latest (0.57.0)

bazel_dep(name = "rules_go", version = "0.55.1")

yeah, that should be the way to go 👍

relates to bazelbuild/bazel-watcher#794

@chenrui333 chenrui333 changed the title ibazel: pin go to [email protected] ibazel: update rules_go in test Sep 4, 2025
@chenrui333 chenrui333 marked this pull request as ready for review September 4, 2025 02:16
@chenrui333 chenrui333 enabled auto-merge September 4, 2025 02:25
@chenrui333 chenrui333 added this pull request to the merge queue Sep 4, 2025
Merged via the queue into main with commit 6f9050a Sep 4, 2025
30 checks passed
@chenrui333 chenrui333 deleted the ibazel-test branch September 4, 2025 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI-no-bottles Merge without publishing bottles go Go use is a significant feature of the PR or issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants