Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Add new arity for table to use default database #39

Merged
merged 1 commit into from
Jun 29, 2015
Merged

Add new arity for table to use default database #39

merged 1 commit into from
Jun 29, 2015

Conversation

danielcompton
Copy link
Collaborator

This is helpful for using an implicit database specified in the connection, ala #38.

@yurrriq
Copy link
Contributor

yurrriq commented Jun 19, 2015

👍 for this

@danielcompton
Copy link
Collaborator Author

It's a bit hard to test this without #38, and it's a bit hard to work on #38 without this :). I'll merge this now and circle back with tests once #38 is finished.

danielcompton added a commit that referenced this pull request Jun 29, 2015
@danielcompton
Merge pull request #39 from danielcompton/table-arity
95ea74e 
Add new arity for table to use default database
@danielcompton danielcompton merged commit 95ea74e into apa512:master Jun 29, 2015
@danielcompton danielcompton deleted the table-arity branch June 29, 2015 04:08
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@danielcompton @yurrriq