Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

convenience procs for db_mysql [#3217] #3318

Closed
wants to merge 5 commits into from
Closed

convenience procs for db_mysql [#3217] #3318

wants to merge 5 commits into from

Conversation

Senketsu
Copy link

Since i'm working on a project that use mysql quite alot and i noticed this (#3217) issue, here is a pull request.
Made this convenience procs , dunno if its what you imagined.

[Please don't mind my spelling.]
Cheers

@Araq
Copy link
Member

Araq commented Sep 14, 2015

This is getting out of hand, 3 different row types? Perhaps your extensions should be in a separate module and if it cannot be, modify db_mysql so that it can be.

@Senketsu
Copy link
Author

Something like this ?

@Araq
Copy link
Member

Araq commented Sep 16, 2015

Yes. Now make it a Nimble package and move it out of the stdlib please. :-)

@Senketsu
Copy link
Author

Edit: Its fine now. Can be merged if no objections.

@Senketsu
Copy link
Author

Senketsu commented Oct 3, 2015

canceling pull request

@Senketsu Senketsu closed this Oct 3, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants