-
Notifications
You must be signed in to change notification settings - Fork 48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bazel/Blaze BUILD #72
Comments
Friendly bump! (Was brought back here by updating to match your changes to third_party). |
Nick, I'm going to tag you, if that's ok, since it looks like you've been rolling out a bunch of great infra commits here @ncooke3 |
Hi @cpsauer, we are willing to consider community support from a PR with an implementation that includes docs and GitHub Action tests. I think a simple workflow to perform build tests on the library targets would be sufficient. |
At a quick glance, your code above looks good. I'm not too familiar with bazel, but would be happy to answer questions where I can. |
Hey, Nick! Thanks for taking the time to reply. |
Hey Googlers!
Any chance you'd be down to release the Blaze/Bazel (Blazel!) BUILD files for this code? Would be handy to those of us building with Bazel, and I assume you've got them internally already. I think the only external dep, PromisesObjC, already has released theirs, which should keep things pretty easy.
If useful, here's the start of one I spun up for the subspecs I needed:
Thanks for your consideration!
Chris
(ex-Googler)
The text was updated successfully, but these errors were encountered: