Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29987559: Noissue - Add "ajv" peer-dependency (Closed)

Created:
Jan. 22, 2019, 1 p.m. by tlucas
Modified:
Jan. 22, 2019, 4:15 p.m.
Reviewers:
Sebastian Noack, kzar
Visibility:
Public.

Description

Noissue - Add "ajv" peer-dependency

Patch Set 1 #

Patch Set 2 : #

Total comments: 6
Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M package.json View 1 1 chunk +3 lines, -1 line 6 comments Download

Messages

Total messages: 6
tlucas
This should get rid of the cron-warnings about "ajv".
Jan. 22, 2019, 1 p.m. (2019-01-22 13:00:59 UTC) #1
tlucas
Patch Set 2: Fixed a typo.
Jan. 22, 2019, 1:02 p.m. (2019-01-22 13:02:28 UTC) #2
kzar
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 ...
Jan. 22, 2019, 4:04 p.m. (2019-01-22 16:04:47 UTC) #3
tlucas
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 ...
Jan. 22, 2019, 4:08 p.m. (2019-01-22 16:08:00 UTC) #4
kzar
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 ...
Jan. 22, 2019, 4:12 p.m. (2019-01-22 16:12:36 UTC) #5
kzar
Jan. 22, 2019, 4:12 p.m. (2019-01-22 16:12:45 UTC) #6
LGTM

Powered by Google App Engine
This is Rietveld