swift-syntax icon indicating copy to clipboard operation
swift-syntax copied to clipboard

Fix Bazel build

Open SimplyDanny opened this issue 2 months ago • 8 comments

Building with Bazel doesn't work at the moment. That's mainly due to the missing dependencies for SwiftRefactorTest and SwiftSyntaxMacros. However, there is also an issue with rules_python which gets fixed by the update of the Bazel version.

To alleviate the risk of causing an outdated Bazel configuration again, I also added an action that at least builds everything in PRs.

SimplyDanny avatar Oct 31 '25 16:10 SimplyDanny

@swift-ci please test

keith avatar Oct 31 '25 16:10 keith

Probably need to clone the repo in the build job as well ... 🤦‍♂️

SimplyDanny avatar Oct 31 '25 16:10 SimplyDanny

Could you please approve the workflow once more, @keith?

SimplyDanny avatar Nov 04 '25 07:11 SimplyDanny

Started

keith avatar Nov 04 '25 17:11 keith

@swift-ci please test

keith avatar Nov 04 '25 20:11 keith

cc @ahoppen is it ok to add a non-blocking job for this?

keith avatar Nov 04 '25 21:11 keith

I’m personally fine with running the Bazel tests in GitHub Actions (thanks for adding them) but I’d like to hear @bnbarham’s opinion as well before merging.

ahoppen avatar Nov 04 '25 22:11 ahoppen

@swift-ci please test

bnbarham avatar Nov 10 '25 23:11 bnbarham