quicktest icon indicating copy to clipboard operation
quicktest copied to clipboard

name the package qt?

Open mvdan opened this issue 4 years ago • 5 comments
trafficstars

I was just thinking that the package layout in this module is a bit weird. I understand that we want the uses of the API to be short, so qt.Assert is nicer than quicktest.Assert. I also get that a module/repo path like github.com/frankban/qt would be a bit confusing.

That said, I think it's a bit unfortunate that we don't just publish the API under a package qt. This would be tricky to do in v1 without duplicating the entire API, but for v2 we could perhaps use something like github.com/frankban/quicktest/v2/qt. The import gets a tiny bit longer, but we avoid having to name the import. Plus, other tools like goimports and pkg.go.dev will work much better, as there will no longer be a mismatch between the actual package name and how it's used in code.

mvdan avatar Feb 22 '21 15:02 mvdan

For v2, the plan is to move to github.com/go-quicktest/qt.

rogpeppe avatar Feb 22 '21 18:02 rogpeppe

Ah, great :)

mvdan avatar Feb 22 '21 18:02 mvdan

Has anyone floated a proposal to integrate Checker into the stdlib testing package? It really feels like it belongs there.

toffaletti avatar Feb 19 '22 07:02 toffaletti

Is v2 still in the works? I'm tired of fighting gopls every time I want to use qt.

josharian avatar Jul 20 '22 21:07 josharian

https://github.com/go-quicktest/qt has been public for some time, but it's not quite ready yet - there is no stable version, for example. It's up to Roger and Francesco.

mvdan avatar Jul 20 '22 21:07 mvdan

Closing for grouping all the qt related considerations at https://github.com/go-quicktest/qt/issues/7

frankban avatar Sep 19 '22 08:09 frankban