import: Use micro-editor/json5 instead of zyedidia/json5 by JoeKar · Pull Request #3595 · micro-editor/micro (original) (raw)

Just a note: I see you dropped the commit zyedidia/json5@2da050b, I don't know intentionally or not, but as a result you added pulling of this https://github.com/robertkrimen/otto repo which is apparently a JS interpreter (so no wonder it is 1.5MB size, larger than any other micro's external dependencies, including gopher-lua which is only 1MB), which we certainly don't need in micro.

Well, I've checked that it doesn't seem to increase the size of the micro binary (it is the same with and without your PR), so the issue is only in needless downloading and storing of this 1.5MB repo (if the user builds micro from source), which is probably not a big deal if it helps to minimize downstream changes.