Skip to content

I was getting errors following the examples here, so I browsed the issues... #27

Merged
merged 1 commit into from Apr 28, 2012

2 participants

@TeeJayGreen

... for clues. Then I found out the order of the parameters (name, type) had been swapped in addParameter(). I think I fixed them all, but I wasn't sure about line 88. Here's the comment saying they were swapped: #14 (comment)

@TeeJayGreen TeeJayGreen I was getting errors following the examples here, so I browsed the is…
…sues for clue. Then I found out the order of the parameters (name, type) had been swapped in addParameter(). I think I fixed them all, but I wasn't sure about line 88. Comment saying they were swapped: pekim#14 (comment)
aaabcc8
@pekim pekim merged commit 38153c1 into pekim:gh-pages Apr 28, 2012
@pekim
Owner
pekim commented Apr 28, 2012

Thank you very much for this.

@pekim
Owner
pekim commented Apr 28, 2012

Line 88 should be ok. It matches the integration test at https://github.com/pekim/tedious/blob/master/test/integration/parameterised-statements-test.coffee#L182, and the tests in that file all pass.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.