Noissue - Add "ajv" peer-dependency

Noissue - Add "ajv" peer-dependency

[tlucas, 2019-01-22 13:00:59]

This should get rid of the cron-warnings about "ajv".

[tlucas, 2019-01-22 13:02:28]

Patch Set 2:
Fixed a typo.

[kzar, 2019-01-22 16:04:47]

https://codereview.adblockplus.org/29987559/diff/29987562/package.json
File package.json (right):

https://codereview.adblockplus.org/29987559/diff/29987562/package.json#newcode7
package.json:7: "'acorn' and 'ajv' are peer dependencies of manifoldjs, we don't
use",
Did you mean to append the comma? I figure you might have intended to
concatenate the strings instead?

https://codereview.adblockplus.org/29987559/diff/29987562/package.json#newcode7
package.json:7: "'acorn' and 'ajv' are peer dependencies of manifoldjs, we don't
use",
Aren't these called "transitive dependencies"?

https://codereview.adblockplus.org/29987559/diff/29987562/package.json#newcode11
package.json:11: "ajv": "^5.0.0",
(I have not looked into this ajv dependency, or checked the version. I take your
word for it they're both right.)

[tlucas, 2019-01-22 16:08:00]

https://codereview.adblockplus.org/29987559/diff/29987562/package.json
File package.json (right):

https://codereview.adblockplus.org/29987559/diff/29987562/package.json#newcode7
package.json:7: "'acorn' and 'ajv' are peer dependencies of manifoldjs, we don't
use",
On 2019/01/22 16:04:47, kzar wrote:
> Did you mean to append the comma? I figure you might have intended to
> concatenate the strings instead?

I did, i wanted to stick to 80 characters.

for the transitivity: not entirely, transitive dependencies get pull it
automatically, peers unfortunately not (hence the review)

https://codereview.adblockplus.org/29987559/diff/29987562/package.json#newcode11
package.json:11: "ajv": "^5.0.0",
On 2019/01/22 16:04:47, kzar wrote:
> (I have not looked into this ajv dependency, or checked the version. I take
your
> word for it they're both right.)

Quoting cron: "npm WARN ajv-keywords@2.1.1 requires a peer of ajv@^5.0.0 but
none was installed"

[kzar, 2019-01-22 16:12:36]

https://codereview.adblockplus.org/29987559/diff/29987562/package.json
File package.json (right):

https://codereview.adblockplus.org/29987559/diff/29987562/package.json#newcode7
package.json:7: "'acorn' and 'ajv' are peer dependencies of manifoldjs, we don't
use",
On 2019/01/22 16:08:00, tlucas wrote:
> for the transitivity: not entirely, transitive dependencies get pull 
> it automatically, peers unfortunately not (hence the review)

Ah, gotya. Good to know, thanks.

[kzar, 2019-01-22 16:12:45]

LGTM