Alex Eagle

Results 412 comments of Alex Eagle

Sorry for the long delay. That seems pretty reasonable to me, except for a concern about placing an annotation in every place we currently have the `//BUG` comment - are...

Hey @jin should we file an issue in bazelbuild/bazel on the repository_ctx.download function? Or do you think `experimental_downloader_config` is a suitable place for the 404 suppression to go?

Looks like a newer version is required. ``` % docker run --rm -it --entrypoint=/bin/sh python:3.8.13 # pip install --only-binary=:all: pycryptodome==3.8.0 ERROR: Could not find a version that satisfies the requirement...

It seems to me that bazel-diff is relying on the **definition** of a repository rule to understand the dependency graph, rather than the result of **executing** the repository rule. In...

Thanks for the tip @achew22 !

you have two different problems 1. We copy all inputs to the output tree, but Bazel only lets us copy within the same Bazel package. So the prettierrc.json file should...

https://github.com/aspect-build/bazel-lib/pull/175 and https://github.com/aspect-build/rules_js/pull/253 provide better error messages.

With https://github.com/bazelbuild/bazel/pull/15781 landed today, we think the only remaining issue with Bazel at HEAD preventing rules_js on RBE is https://github.com/bazelbuild/bazel/issues/14224

https://github.com/aspect-build/rules_js/pull/400 shows that rules_js is now working with RBE at HEAD! However it requires one change to Bazel which is on this fork: https://github.com/aspect-build/bazel/releases/tag/6.0.0-aspect1 https://github.com/aspect-build/rules_js/blob/main/.github/workflows/ci.yaml#L56 shows where we select our...

For anyone who wants to try RBE with rules_js or rules_ts now, here are the steps: 1. Decide where you want to enable this. Since there is no release of...