fix(linting): no-unused-vars

Author: lukekarrys

lib/read-json.js

@@ -27,7 +27,7 @@ readJson.extraSet = [
 var typoWarned = {}
 var cache = {}
 
-function readJson (file, log_, strict_, cb_) {
+function readJson (file) {
   var log, strict, cb
   for (var i = 1; i < arguments.length - 1; i++) {
     if (typeof arguments[i] === 'boolean') {
@@ -154,7 +154,7 @@ function indexjs (file, er, log, strict, cb) {
 }
 
 readJson.extras = extras
-function extras (file, data, log_, strict_, cb_) {
+function extras (file, data) {
   var log, strict, cb
   for (var i = 2; i < arguments.length - 1; i++) {
     if (typeof arguments[i] === 'boolean') {

test/bin.js

@@ -5,7 +5,7 @@ var tap = require('tap')
 var readJson = require('../')
 
 var createWarningCollector = function () {
-  var warn = function (msg) {
+  var warn = function () {
     warn.warnings.push(arguments)
   }
   warn.warnings = []
@@ -25,7 +25,7 @@ tap.test('Bin test', function (t) {
 tap.test('Bad bin test', function (t) {
   var p = path.resolve(__dirname, 'fixtures/badbin.json')
   var warn = createWarningCollector()
-  readJson(p, warn, function (er, data) {
+  readJson(p, warn, function () {
     t.equal(warn.warnings.length, 1)
     t.equal(warn.warnings[0][2], 'No bin file found at bin/typo')
     t.end()

test/helpful.js

@@ -4,15 +4,15 @@ var path = require('path')
 var p = path.resolve(__dirname, 'fixtures/erroneous.json')
 
 tap.test('erroneous package data', function (t) {
-  readJson(p, function (er, data) {
+  readJson(p, function (er) {
     t.ok(er instanceof Error)
     t.equal(er.code, 'EJSONPARSE')
     t.end()
   })
 })
 
 tap.test('ENOTDIR for non-directory packages', function (t) {
-  readJson(path.resolve(__filename, 'package.json'), function (er, data) {
+  readJson(path.resolve(__filename, 'package.json'), function (er) {
     t.ok(er)
     t.equal(er.code, 'ENOTDIR')
     t.end()

test/main-non-string.js

@@ -6,7 +6,7 @@ var p = path.resolve(__dirname, 'fixtures/badmainnonstring.json')
 tap.test('non-string main entries', function (t) {
   var logmsgs = []
   const warn = (...msg) => logmsgs.push(msg)
-  readJson(p, warn, function (er, data) {
+  readJson(p, warn, function (er) {
     t.comment(logmsgs.map(msg => 'Warning: ' + msg.join(' ')).join('\n'))
     t.match(er, new TypeError('The "main" attribute must be of type string.'))
     t.end()