bazelbuild / reclient

Apache License 2.0
66 stars 17 forks source link

External Contribution Status #51

Closed daniel-sudz closed 2 months ago

daniel-sudz commented 3 months ago

I have been stalled on getting this repo to build on windows and mac for a while and I want to know how to resolve this moving forward. There have been some fixes merged here and there but I would ideally want to be able to contribute PRs directly to have a faster turnaround. @bentekkie

gkousik commented 3 months ago

Hi, Thanks for raising this - I'm working on setting up a workflow so that we can import Github PRs. Will update on this in a week.

daniel-sudz commented 3 months ago

Hi, Thanks for raising this - I'm working on setting up a workflow so that we can import Github PRs. Will update on this in a week.

Thanks for the consideration. Would it be possible to also consider one of the following:

It's really hard to report or even communicate any issue when it's not particularly clear what the problem is. Ie. the most recent commit 4bdf44f prevents me from resolving "@io_bazel_rules_go" and I can't tell if this is a local setup issue or something else.

Thanks again!

gkousik commented 3 months ago

Hi thanks again for the followup, we are working on setting up a public build integration through Github Actions (expect an update on that too in ~a week).

daniel-sudz commented 3 months ago

thank you so much to everyone that resolved this :smile:

gkousik commented 3 months ago

Sorry, there's one final component here (its to update the CONTRIBUTING.md file). I'm waiting on some approvals to do that, I'll keep this open until then.

daniel-sudz commented 3 months ago

Sorry, there's one final component here (its to update the CONTRIBUTING.md file). I'm waiting on some approvals to do that, I'll keep this open until then.

thanks, when that gets added I have a first trial run for you: https://github.com/bazelbuild/reclient/pull/60 :smiley:

gkousik commented 2 months ago

The wheels have turned and I've updated the CONTRIBUTING.md page. Closing this one.