-
Notifications
You must be signed in to change notification settings - Fork 371
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
Rewrite blackbox in Go (golang) #310
Comments
The "golang" branch has made a lot of progress this weekend:
The code is A LOT more simple now. This is the 3rd (?) iteration of trying to simplify the code. It is a lot better than my earlier iterations. The interfaces are much better structured now. I think anyone should be able to hack on this very easily. In fact, it is so much cleaner that it requires less test code. If you want to see the code so far, it is in this branch: CC: @farseeker |
Beta implementation is in #250 |
No Go tests yet, right? |
I had to dig through some notes.
The go tests are:
Unit tests:
go test ./...
Integration tests:
cd integrationTest && rm -rf /tmp/bbhome-* && go test -long --nocleanup
Tom
…On Wed, Nov 11, 2020 at 4:53 PM Max Horstmann ***@***.***> wrote:
No Go tests yet, right?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#310 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABQATHJHU37ETUYYNRDUHEDSPMBXJANCNFSM4NXWVQCA>
.
|
This is the issue to track the development of a Go version.
CC: @hpoul @mnewswanger @MaxHorstmann @captncraig
The text was updated successfully, but these errors were encountered: