refactor!: remove package.json-based parserConfiguration

test/fixtures/no-extension

@@ -4,4 +4,7 @@
 
 var parser = require('../../yargs.js')(process.argv.slice(2))
 
-console.log(parser.argv)
+console.log(parser.parserConfiguration({
+    'dot-notation': false,
+    'boolean-negation': false
+}).argv)

test/fixtures/no-require-main.js

@@ -7,4 +7,7 @@ delete require.main
 
 var parser = require('../../yargs.js')(process.argv.slice(2), undefined, require)
 
-console.log(parser.argv)
+console.log(parser.parserConfiguration({
+    'dot-notation': false,
+    'boolean-negation': false
+}).argv)

test/fixtures/normal-bin.js

@@ -1,6 +1,10 @@
 #!/usr/bin/env node
-var argv = require('./yargs/index.js')
+var argv = require('../../index.js')
   .help('help')
   .version()
+  .parserConfiguration({
+    'dot-notation': false,
+    'boolean-negation': false
+  })
   .argv
 console.log(argv)

test/fixtures/package.json

@@ -2,9 +2,5 @@
   "version": "9.9.9",
   "repository": {
     "type": "svn"
-  },
-  "yargs": {
-    "dot-notation": false,
-    "boolean-negation": false
   }
 }

test/fixtures/symlink-bin.js

@@ -2,5 +2,9 @@
 var argv = require('./yargs-symlink/index.js')
   .help('help')
   .version()
+  .parserConfiguration({
+    'dot-notation': false,
+    'boolean-negation': false
+  })
   .argv
 console.log(argv)

test/integration.js

@@ -191,18 +191,6 @@ describe('integration tests', () => {
             return done()
           })
         })
-
-        it('is overridden by yargs.parserConfiguration', (done) => {
-          testCmd('./configured-bin.js', [ '--foo.bar', '--no-baz' ], (code, stdout) => {
-            if (code) {
-              return done(new Error(`cmd exited with code ${code}`))
-            }
-
-            stdout.should.not.match(/foo\.bar/)
-            stdout.should.match(/noBaz/)
-            return done()
-          })
-        })
       })
 
       after(() => {

yargs.js

@@ -1,4 +1,5 @@
 'use strict'
+
 const argsert = require('./lib/argsert')
 const fs = require('fs')
 const Command = require('./lib/command')
@@ -603,7 +604,7 @@ function Yargs (processArgs, cwd, parentRequire) {
 
       const demand = opt.demand || opt.required || opt.require
 
-      // deprecated, use 'demandOption' instead
+      // A required option can be specified via "demand: true".
       if (demand) {
         self.demand(key, demand)
       }
@@ -1032,13 +1033,6 @@ function Yargs (processArgs, cwd, parentRequire) {
     options.__ = y18n.__
     options.configuration = self.getParserConfiguration()
 
-    // Deprecated
-    let pkgConfig = pkgUp()['yargs']
-    if (pkgConfig) {
-      console.warn('Configuring yargs through package.json is deprecated and will be removed in a future major release, please use the JS API instead.')
-      options.configuration = Object.assign({}, pkgConfig, options.configuration)
-    }
-
     const parsed = Parser.detailed(args, options)
     let argv = parsed.argv
     if (parseContext) argv = Object.assign({}, argv, parseContext)