Skip to content

Conversation

@fmeum
Copy link
Contributor

@fmeum fmeum commented Jun 8, 2022

This is highly experimental and thus carries a print warning.

@fmeum fmeum force-pushed the rules-go-0.33.0 branch 4 times, most recently from 3fbe8b5 to 43960d2 Compare June 9, 2022 08:30
@uhthomas
Copy link

This is cool -- what else is there to do to get this working?

@fmeum fmeum force-pushed the rules-go-0.33.0 branch 2 times, most recently from fde55be to 8794b37 Compare July 1, 2022 17:20
This is highly experimental and thus carries a `print` warning.
@fmeum fmeum force-pushed the rules-go-0.33.0 branch from 8794b37 to 6b9d409 Compare July 1, 2022 17:24
@fmeum fmeum marked this pull request as ready for review July 1, 2022 17:26
@fmeum
Copy link
Contributor Author

fmeum commented Jul 1, 2022

This is cool -- what else is there to do to get this working?

It should be working now :-)

Once rules_go is in the registry, I will try to get my prototype integration for bazel-gazelle in, which should then allow rules_go to fetch the dependencies it needs for proto support.

@fmeum fmeum requested a review from alexeagle July 1, 2022 17:29
@uhthomas
Copy link

uhthomas commented Jul 1, 2022

Amazing! Thanks so much for doing this. I wrote my own module for rules_go before I realised this PR was open and I feel a bit silly :)

Looking forward to getting this merged and further additions to the BCR.

@meteorcloudy
Copy link
Member

This is just awesome, thanks again @fmeum !

@fmeum
Copy link
Contributor Author

fmeum commented Jul 4, 2022

@meteorcloudy Is it possible that you forgot to hit the merge button? :)

Copy link
Member

@Wyverald Wyverald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! :)

@Wyverald Wyverald merged commit 6ce5eea into bazelbuild:main Jul 4, 2022
@fmeum fmeum deleted the rules-go-0.33.0 branch July 4, 2022 15:23
@meteorcloudy
Copy link
Member

Is it possible that you forgot to hit the merge button? :)

Yes, sorry about that..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants