sqlx
sqlx copied to clipboard
[issue-447] added NamedQueryContext method on sqlx.Tx
Issue-447 added NamedQueryContext method on *sqlx.Tx
Pull Request Test Coverage Report for Build 117
- 3 of 3 (100.0%) changed or added relevant lines in 1 file are covered.
- No unchanged relevant lines lost coverage.
- Overall coverage increased (+0.7%) to 67.689%
Totals | |
---|---|
Change from base Build 113: | 0.7% |
Covered Lines: | 1986 |
Relevant Lines: | 2934 |
💛 - Coveralls
@jmoiron any reason not to move ahead with this one? Similar PRs have been raised before without any feedback. Not having this in places means that the transaction type can't be used in places where the database can
Why is this one not merged?
This one would be useful
@jmoiron please merge this PR, without it it's impossible to use this functionality in abstractions over *DB and *Tx