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

doc: fix spelling mistakes #868

Merged
merged 4 commits into from Jul 31, 2017

Conversation

Projects
None yet
3 participants
@davecramer
Member

davecramer commented Jul 17, 2017

No description provided.

@codecov-io

This comment has been minimized.

codecov-io commented Jul 17, 2017

Codecov Report

Merging #868 into master will increase coverage by 0.07%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #868      +/-   ##
============================================
+ Coverage     65.64%   65.71%   +0.07%     
- Complexity     3544     3546       +2     
============================================
  Files           166      166              
  Lines         15252    15253       +1     
  Branches       2474     2474              
============================================
+ Hits          10012    10024      +12     
+ Misses         4061     4051      -10     
+ Partials       1179     1178       -1
@vlsi

This comment has been minimized.

Member

vlsi commented Jul 17, 2017

@davecramer, would you please fix "trucated" in README.md as well?

@@ -15,7 +15,7 @@ The driver supports the nesting and the mixing of escaped functions and escaped
values. The appendix C of the JDBC specification describes the functions.
Some functions in the following tables are translated but not reported as supported
because they are duplicating or changing ther order of the arguments. While this
because they are duplicating or changing theirr order of the arguments. While this

This comment has been minimized.

@vlsi

vlsi Jul 23, 2017

Member

theirr -> their ?

@@ -86,7 +86,7 @@ class Listener extends Thread
if (notifications != null)
{
for (int i=0; i<notifications.length; i++)
for (int i=0; i<;notifications.length; i++)

This comment has been minimized.

@vlsi

vlsi Jul 23, 2017

Member

i<;notifications.length -> i<notifications.length ? (remove extra ;)

@vlsi vlsi changed the title from fix truncated spelling mistake to doc: fix spelling mistakes Jul 31, 2017

@vlsi vlsi added this to the 42.1.4 milestone Jul 31, 2017

@vlsi vlsi merged commit 757db62 into pgjdbc:master Jul 31, 2017

2 checks passed

codecov/project 65.71% (+0.07%) compared to c1d743f
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

davecramer added a commit to davecramer/pgjdbc that referenced this pull request Sep 19, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment