Skip to content
This repository has been archived by the owner on Feb 15, 2023. It is now read-only.

btuan/count #230

Merged
merged 5 commits into from
Apr 1, 2019
Merged

btuan/count #230

merged 5 commits into from
Apr 1, 2019

Conversation

btuan
Copy link
Contributor

@btuan btuan commented Mar 29, 2019

Implements a basic Count receiver that wraps SELECT COUNT(*) functionality in SQL databases.

@coveralls
Copy link

coveralls commented Mar 29, 2019

Pull Request Test Coverage Report for Build 1365

  • 50 of 68 (73.53%) changed or added relevant lines in 3 files are covered.
  • 10 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.02%) to 62.873%

Changes Missing Coverage Covered Lines Changed/Added Lines %
sqlgen/db.go 15 23 65.22%
sqlgen/reflect.go 22 32 68.75%
Files with Coverage Reduction New Missed Lines %
graphql/schemabuilder/input.go 10 61.61%
Totals Coverage Status
Change from base Build 1343: 0.02%
Covered Lines: 3646
Relevant Lines: 5799

💛 - Coveralls

@btuan btuan requested a review from ehzhang March 29, 2019 05:18
sqlgen/db.go Outdated Show resolved Hide resolved
sqlgen/db.go Outdated Show resolved Hide resolved
@btuan btuan force-pushed the btuan/count branch 2 times, most recently from 3f31e7c to cca032a Compare March 29, 2019 17:52
sqlgen/db.go Outdated Show resolved Hide resolved
@berfarah
Copy link
Contributor

hype

@btuan btuan force-pushed the btuan/count branch 2 times, most recently from 095d98d to dfc6b16 Compare March 29, 2019 21:14
sqlgen/reflect.go Outdated Show resolved Hide resolved
Copy link
Contributor

@berfarah berfarah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From offline chat:
Trade-off is being consistent with the code-base as is vs making this library easy to maintain going forward. I favor the latter by exporting as little as possible so we can change things around if we need to. In this case, that means exporting a SQLQuery interface.

Copy link
Contributor

@berfarah berfarah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❤️

@btuan
Copy link
Contributor Author

btuan commented Mar 30, 2019

I plan on merging this Monday, barring any additional comments!

@btuan btuan merged commit 18d6ed0 into master Apr 1, 2019
@btuan btuan deleted the btuan/count branch April 1, 2019 15:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants