c14h, yarpnarp: sql.go refactoring #7
Labels
Comments
Merovius
added a commit
that referenced
this issue
Jul 30, 2016
The data package should improve code-reuse by bundling together the SQL operations into a common package. It will be marginally less efficient than using SQL directly, but it will enforce best practices, remove duplication and prevent mistakes from broken queries. #7 is the associated issue. To begin with, this updates termine to use the new package. It also refactors termine a bit. Fixes #3
cherti
pushed a commit
that referenced
this issue
Jul 30, 2016
* Add a package to abstract SQL operations The data package should improve code-reuse by bundling together the SQL operations into a common package. It will be marginally less efficient than using SQL directly, but it will enforce best practices, remove duplication and prevent mistakes from broken queries. #7 is the associated issue. To begin with, this updates termine to use the new package. It also refactors termine a bit. Fixes #3
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
sql.go sollte refactored werden, um http://github.com/jmoiron/sqlx zu verwenden. Ich würde vmtl. zwei Schritte vorschlagen:
Wir sollten auch das sql-CRUD in ein eigenes Paket auslagern, welches dann von allen commands gemeinsam genutzt werden kann.
The text was updated successfully, but these errors were encountered: