Skip to content

stringArrayParser can parse empty arrays + stringArrayParser tests #58

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

Merged
merged 1 commit into from
Oct 24, 2011

Conversation

tokumine
Copy link

Hi there,

I updated the stringArrayParser so that it can also parse empty postgres arrays. Also added some tests for the parser in general.

Thanks for the great library!

@brianc brianc merged commit 53423df into brianc:master Oct 24, 2011
@brianc
Copy link
Owner

brianc commented Oct 24, 2011

thank you!

On Mon, Oct 24, 2011 at 8:09 AM, Simon Tokumine <
reply@reply.github.com>wrote:

Hi there,

I updated the stringArrayParser so that it can also parse empty postgres
arrays. Also added some tests for the parser in general.

Thanks for the great library!

You can merge this Pull Request by running:

git pull https://github.com/tokumine/node-postgres master

Or you can view, comment on it, or merge it online at:

#58

-- Commit Summary --

  • stringArrayParser can parse empty arrays & stringArrayParser tests

-- File Changes --

M lib/types.js (1)
M test/unit/client/query-tests.js (30)

-- Patch Links --

https://github.com/brianc/node-postgres/pull/58.patch
https://github.com/brianc/node-postgres/pull/58.diff

Reply to this email directly or view it on GitHub:
#58

brianc added a commit that referenced this pull request Dec 18, 2019
Add additional pool test & deprecate .end
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