search

brianc / node-pg-types

Type parsing for node-postgres
267 stars 54 forks source link

Remove null checks #122

Closed charmander closed 3 years ago

charmander commented 3 years ago

They’re applied inconsistently (not all parsers include them) and can easily be done by the caller (like pg already does).

codecov-io commented 3 years ago

Codecov Report

Merging #122 into master will decrease coverage by 0.47%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #122      +/-   ##
==========================================
- Coverage   91.70%   91.23%   -0.48%     
==========================================
  Files           4        4              
  Lines         205      194      -11     
==========================================
- Hits          188      177      -11     
  Misses         17       17
Impacted Files Coverage Δ
lib/binaryParsers.js 86.11% <ø> (-0.20%) :arrow_down:
lib/textParsers.js 98.01% <ø> (-0.18%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 8477d7c...3c8aa72. Read the comment docs.