search

lmco / laikaboss

Laika BOSS: Object Scanning System
Apache License 2.0
739 stars 156 forks source link

Add -j option #12

Closed jshlbrd closed 9 years ago

jshlbrd commented 9 years ago

This pull requests adds a -j option to log JSON results to a file path.

csmutz commented 9 years ago

jshlbrd,

Excuse relatively slow response. We appreciate your contribution.

We plan to merge this change. Many people achieve the same end by simply using shell redirection, but we'll add this method as well.

If you'd like you or your organization added to the copyright notice, please just add another copyright notice at the top of the file.

Otherwise, we'll merge this soon.

jshlbrd commented 9 years ago

I don't think adding a Copyright notice is necessary, but if you would consider making a contributors file in the repo and adding me to that, then that would be fine.

On Tue, Sep 15, 2015 at 9:44 AM, csmutz notifications@github.com wrote:

jshlbrd,

Excuse relatively slow response. We appreciate your contribution.

We plan to merge this change. Many people achieve the same end by simply using shell redirection, but we'll add this method as well.

If you'd like you or your organization added to the copyright notice, please just add another copyright notice at the top of the file.

Otherwise, we'll merge this soon.

— Reply to this email directly or view it on GitHub https://github.com/lmco/laikaboss/pull/12#issuecomment-140397881.