sgreben / tj

stdin line timestamps. single binary, no dependencies. osx & linux & windows. plays well with jq.
230 stars 2 forks source link

Fix -read-json examples #3

Closed wayneashleyberry closed 6 years ago

wayneashleyberry commented 6 years ago

The readme was showing examples using the -read-json flag, whereas the actual flag is -readjson

sgreben commented 6 years ago

Hey, many thanks for taking the time to do the PR! It indeed used to be -readjson but in 6.0.0 i decided to make the flag names more consistent and thus renamed a bunch of them, among others -readjson -> -read-json. I'm not aware of a way to announce things via github, and as a result this was kind of buried in the release notes for 6.0.0:

Breaking changes:

- Rename -delta-buffer to -match-buffer
- Rename -readjson to -read-json
- Rename -jsontemplate to -match-template
- Rename -start to -regex / -match-regex
- Rename -timeformat to -time-format
- Rename .JSONText to .MatchText
- Replace .StartText, .StartObject by .Start
- Literals (no "{{") given to -match / -match-template are automatically surrounded by {{}}

Additions:
- Add JSON stream parsing (one-object-per-line format no longer required)
- Add -match-condition
- Add -time-zone
- Add -scale-{sqr,cube,sqrt,cubert}
- Add Text and TimeColor preset templates

Internal:
- Refactor main to avoid duplicating logic between JSON/line input

Unless I'm missing something, it's still -read-json in the code - would it be fine with you to close this issue?

wayneashleyberry commented 6 years ago

Ah yes, I updated on my side and the new flags work just as documented.

An aside, the -version flag isn't printing anything :/