You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I just start using pgsql and I like it. Good functionality and it seems to be pretty well documented. So thank you for that =)
But I don't like the directory layout introduced in commit (6cf4313). It's really cumbersome when installed with goinstall, and in general it's a little more than necessary.
If installed with goinstall, the import path in Go source files is really complicated and hard to remember.
import"github.com/lxn/go-pgsql/src/pkg/pgsql"
(It doesn't really make since to have a directory /src/ in the import path either, because you're importing a binary file.)
The source files (in /src/pkg/pgsql) should be moved to /pgsql. It is the Go convention. So it gives a shorter, easy to remember import path.
import"github.com/lxn/go-pgsql/pgsql"
I think the examples were good in the /examples directory. The project wouldn't be cluttered, imho.
Please consider this. I think its really annoying the way it is. But otherwise, thanks again for this package! And please keep up the great work!
The text was updated successfully, but these errors were encountered:
I just start using pgsql and I like it. Good functionality and it seems to be pretty well documented. So thank you for that =)
But I don't like the directory layout introduced in commit (6cf4313). It's really cumbersome when installed with
goinstall
, and in general it's a little more than necessary.If installed with
goinstall
, the import path in Go source files is really complicated and hard to remember.(It doesn't really make since to have a directory
/src/
in the import path either, because you're importing a binary file.)The source files (in
/src/pkg/pgsql
) should be moved to/pgsql
. It is the Go convention. So it gives a shorter, easy to remember import path.I think the examples were good in the
/examples
directory. The project wouldn't be cluttered, imho.Please consider this. I think its really annoying the way it is. But otherwise, thanks again for this package! And please keep up the great work!
The text was updated successfully, but these errors were encountered: