On Friday, December 20, 2013 4:43:41 AM UTC-8, Jurgens du Toit wrote:
>
> Hi Jeremy
>
> I managed to get sequel to handle the stored proc with output parameters 
> correctly. The commit I referenced in the previous commit needed only a 
> small tweak to get it working properly.
>
> It does however need some polishing. Would you mind reviewing it and 
> sending me the feedback?
>

It looks OK to me.  It wasn't aware you could hack around lack of support 
in the driver in the manner you did.  In order to get it added to Sequel, 
you should add a test to spec/adapters/mssql_spec.rb.

Thanks,
Jeremy

-- 
You received this message because you are subscribed to the Google Groups 
"sequel-talk" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sequel-talk.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to