[ #53 , JSON output option ]

[mohamedsalem401]

Write output fileObjects to .markdowndb/files.json # array of file objects

[changeset-bot]

🦋 Changeset detected

Latest commit: e8a7dcb

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
mddb	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[rufuspollock]

I'd think we'd want a bit of docs here - both in README.md plus probably on the command line to explain that this happens.

I'd have also kept the code path for generating json separate from sqlite both in code and in command line (i probably don't want both).

[mohamedsalem401]

I'd have also kept the code path for generating json separate from sqlite both in code and in command line (I probably don't want both).

You mean to add an option to generate either SQL or JSON but not both simultaneously? Right?

[rufuspollock]

I'd have also kept the code path for generating json separate from sqlite both in code and in command line (I probably don't want both).

You mean to add an option to generate either SQL or JSON but not both simultaneously? Right?

OK. I'm going to apply YAGNI and for now let's just generate both as you are doing. let's not bother with both paths.

[rufuspollock]

@mohamedsalem401 the one thing to do is probably just to add a changeset and then we can merge as is for now.