tooltitude / support

Tooltitude Support
https://www.tooltitude.com
Other
21 stars 0 forks source link

could not generate interface stub #14

Closed BestNathan closed 1 year ago

BestNathan commented 1 year ago

Tooltitude Version: v0.23.1

OS Version: 22.3.0 Darwin Kernel Version 22.3.0

VS Code Version:1.78.2

Go Version:1.20.3

Code Repository (if open source):

What did you try to do?

Is there anything interesting in the tooltitude output channel? (should be open if you are reporting error from within VS Code)

Steps to reproduce

image

BestNathan commented 1 year ago

when I downgrade to 0.22.5 this problem solved

tooltitude-support commented 1 year ago

@BestNathan Hi! Thanks for reporting! I will take a look at ti!

tooltitude-support commented 1 year ago

@BestNathan Is it possible to share a bit of the structure of your file? On obvious examples, it doesn't reproduce unfortunately.

tooltitude-support commented 1 year ago

@BestNathan Also which interface do you implement? How many methods does it have? Does it have embedded interfaces?

tooltitude-support commented 1 year ago

Ok. I reproduced it. Will fix soon. It's enough to implement Context on a simple struct type.

tooltitude-support commented 1 year ago

Ok. Fixed it. 0.23.2 will be out soon (just launched deployment).

tooltitude-support commented 1 year ago

Closing it since it's fixed.

BestNathan commented 1 year ago

@BestNathan Also which interface do you implement? How many methods does it have? Does it have embedded interfaces?

@tooltitude-support hi bro, I meet something like this.

image

as above, I have A implemented slog.LogValuer and B embeds A, and then when I want to make B implement slog.LogValuer, it doesn't generate the stub. it seems that has already been implemented, but what if I really want B to implement the interface itself, is that an expected result?

tooltitude-support commented 1 year ago

@BestNathan Ok, got your problem. I think in this situation it's logical to include stub implementations so that the user could choose which ones to leave. If VS Code had an API for more complicated dialogs, then it would make sense to present a choice.

tooltitude-support commented 1 year ago

@BestNathan I created a separate issue for your problem.