Gold856

Results 165 comments of Gold856

We'd cache only main for the same reason GitHub Actions has separate caches for different PRs: [preventing cache poisoning](https://github.com/bazelbuild/bazel/issues/4276).

Allowing PRs to share caches still scares me a little (I'm just gonna make a really bad PR and submit it), but PRs are more visible so it's easier to...

It would be nice if we could use Sleipnir instead of Ceres since Sleipnir is already a part of the build, so using Sleipnir means less work for us. Also,...

We can add Ceres as a thirdparty repo and have WPICal pull it in. If that helps maintainability, I’m all for it.

I didn't find any Glass includes, so I think we can remove that from the dependency lists.

Looks like you accidentally force pushed your commits away. Since you updated the main branch, you should be able to reopen the PR.

I've assembled all the dependencies that wpical should need. To reduce merge conflicts, please resolve all review comments first, then I will create a PR on your fork that you...

Blocked until #6934 is fixed.

Closing in favor of switching to Bazel.