Skip to content

Conversation

@sxh519
Copy link

@sxh519 sxh519 commented Aug 17, 2020

fix #334

@sxh519 sxh519 requested a review from a team as a code owner August 17, 2020 03:07
@grumbach grumbach self-requested a review August 19, 2020 14:18
Copy link
Member

@grumbach grumbach left a comment

Choose a reason for hiding this comment

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

Thank you very much for this PR! But could you please adapt the tests for the CI checks before we Merge this? Thanks!

daemon/common_test.go:32:3: cannot use &mockDriver literal (type *mockDriver) as type Driver in return argument:
	*mockDriver does not implement Driver (wrong type for State method)
		have State() (*"github.com/bblfsh/bblfshd/daemon/protocol".DriverInstanceState, error)
		want State() (*"github.com/bblfsh/bblfshd/v2/daemon/protocol".DriverInstanceState, error)
daemon/pool_test.go:127:6: impossible type assertion:
	*mockDriver does not implement Driver (wrong type for State method)
		have State() (*"github.com/bblfsh/bblfshd/daemon/protocol".DriverInstanceState, error)
		want State() (*"github.com/bblfsh/bblfshd/v2/daemon/protocol".DriverInstanceState, error)

@kuba--
Copy link
Member

kuba-- commented Jul 4, 2023

@sxh519 - are you still working on this?
Can you fix tests ☝🏻 ?

@david972 david972 self-requested a review July 5, 2023 07:11
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.

Cannot get latest version: module contains a go.mod file, so module path should be github.com/bblfsh/bblfshd/v2

4 participants