Skip to content

Add Big Number support #40

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.

Sign up for GitHub

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

Open
SebastianG77 wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Add Big Number support #40

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
eba3d70
add BigNumber Support for function diffString()
Feb 8, 2019
abb3d16
add test cases for testing big number support of function colorize()
Feb 8, 2019
955c1c3
pass options.bigNumberSupport to all extendedTypeOf() function calls …
Feb 8, 2019
2703cef
Function JSONbig.stringify() will only be called if bigNumberSupport …
Feb 9, 2019
7b8a2f0
add parameter to activate big number support via cli
Feb 9, 2019
85d0a89
improve cli description for --bigNumberSupport
Feb 9, 2019
1f4461d
add special comparison for BigNumber values
Feb 9, 2019
44e63f4
add several test cases for testing BigNumber Support
Feb 9, 2019
f4c4b9c
WIP: test if Arrays containing BigNumbers and ordinary integers are c…
Feb 9, 2019
e9cf579
fix test case for creating a diff for an array containing big number …
SebastianG77 Feb 10, 2019
4cbc3bd
extend README.md by explaining parameter bigNumberSupport briefly
SebastianG77 Feb 10, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
add parameter to activate big number support via cli
  • Loading branch information
SebastianG committed Feb 9, 2019
commit 7b8a2f0332b1bf93497b6fe66b63f12eeb813a04
10 changes: 6 additions & 4 deletions lib/cli.coffee
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
fs = require 'fs'
JSONbig = require 'true-json-bigint'
tty = require 'tty'

{ diff } = require './index'
Expand All @@ -13,6 +14,7 @@ module.exports = (argv) ->
" second.json New file #var(file2) #required"

"General options:"
" -b, --bigNumberSupport Parse larger numbers to bignumber.js objects"
" -v, --verbose Output progress info"
" -C, --[no-]color Colored output"
" -j, --raw-json Display raw JSON encoding of the diff #var(raw)"
Expand All @@ -26,9 +28,9 @@ module.exports = (argv) ->
data2 = fs.readFileSync(options.file2, 'utf8')

process.stderr.write "Parsing old file...\n" if options.verbose
json1 = JSON.parse(data1)
json1 = if options.bigNumberSupport then JSONbig.parse(data1) else JSON.parse(data1)
process.stderr.write "Parsing new file...\n" if options.verbose
json2 = JSON.parse(data2)
json2 = if options.bigNumberSupport then JSONbig.parse(data2) else JSON.parse(data2)

process.stderr.write "Running diff...\n" if options.verbose
result = diff(json1, json2, options)
Expand All @@ -38,10 +40,10 @@ module.exports = (argv) ->
if result
if options.raw
process.stderr.write "Serializing JSON output...\n" if options.verbose
process.stdout.write JSON.stringify(result, null, 2)
process.stdout.write if options.bigNumberSupport then JSONbig.stringify(result, null, 2) else JSON.stringify(result, null, 2)
else
process.stderr.write "Producing colored output...\n" if options.verbose
process.stdout.write colorize(result, color: options.color)
process.stdout.write colorize(result, {color: options.color, bigNumberSupport: if options.bigNumberSupport then true else false })
else
process.stderr.write "No diff" if options.verbose

Expand Down