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

fix: PgSQLXML setCharacterStream() results in null value #1608

Merged
merged 2 commits into from Nov 15, 2019

Conversation

davecramer
Copy link
Member

fixes Issue #731

@sehrope
Copy link
Member

sehrope commented Nov 14, 2019

@davecramer Need to add this to a suite for it to run.

Also, s/Copyright (c) 2004/Copyright (c) 2019/.

@davecramer
Copy link
Member Author

@sehrope I knew that and got distracted :( ADD kicking in

@AppVeyorBot
Copy link

Build pgjdbc 1.0.514 completed (commit cfe462fa96 by @davecramer)

@sehrope
Copy link
Member

sehrope commented Nov 14, 2019

@davecramer Well it is a part of the PR checklist template ;-)

@AppVeyorBot
Copy link

Build pgjdbc 1.0.515 completed (commit c4130e1c0b by @davecramer)

@davecramer
Copy link
Member Author

@sehrope that's what that's for :)

@codecov-io
Copy link

Codecov Report

Merging #1608 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #1608      +/-   ##
============================================
+ Coverage     68.96%   68.97%   +<.01%     
+ Complexity     4008     4007       -1     
============================================
  Files           179      179              
  Lines         16654    16655       +1     
  Branches       2714     2714              
============================================
+ Hits          11486    11488       +2     
+ Misses         3911     3908       -3     
- Partials       1257     1259       +2

@davecramer davecramer merged commit 1e37026 into pgjdbc:master Nov 15, 2019
@davecramer davecramer deleted the fixpgsqlxml branch November 15, 2019 12:42
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.

None yet

4 participants