update the option to be part of an object

Author: deecewan

docs/rules/jsx-no-literals.md

@@ -23,12 +23,12 @@ var Hello = <div>{'test'}</div>;
 
 There is only one option:
 
-* `no-strings` - Enforces no string literals used as children, wrapped or unwrapped.
+* `noStrings` - Enforces no string literals used as children, wrapped or unwrapped.
 
 To use, you can specify like the following:
 
 ```json
-"react/jsx-no-literals": ["no-strings"]
+"react/jsx-no-literals": [{"noStrings": false}]
 ```
 
 In this configuration, the following are considered warnings:

lib/rules/jsx-no-literals.js

@@ -9,8 +9,6 @@
 // Rule Definition
 // ------------------------------------------------------------------------------
 
-var NO_STRINGS = 'no-strings';
-
 module.exports = {
   meta: {
     docs: {
@@ -20,17 +18,18 @@ module.exports = {
     },
 
     schema: [{
-      enum: [NO_STRINGS]
-    }, {
       type: 'object',
-      properties: {},
+      properties: {
+        noStrings: {
+          type: 'boolean'
+        }
+      },
       additionalProperties: false
     }]
   },
 
   create: function(context) {
-
-    const isNoStrings = context.options[0] === NO_STRINGS;
+    const isNoStrings = context.options[0] ? context.options[0].noStrings : false;
 
     const message = isNoStrings ?
       'Strings not allowed in JSX files' :

tests/lib/rules/jsx-no-literals.js

@@ -117,29 +117,29 @@ ruleTester.run('jsx-no-literals', rule, {
         </Foo>
       `,
       parser: 'babel-eslint',
-      options: ['no-strings']
+      options: [{noStrings: true}]
     }, {
       code: `
         <Foo bar="test">
           {translate('my.translate.key')}
         </Foo>
       `,
       parser: 'babel-eslint',
-      options: ['no-strings']
+      options: [{noStrings: true}]
     }, {
       code: `
         <Foo bar="test">
           {intl.formatText(message)}
         </Foo>
       `,
-      options: ['no-strings']
+      options: [{noStrings: true}]
     }, {
       code: `
         <Foo bar="test">
           {translate('my.translate.key')}
         </Foo>
       `,
-      options: ['no-strings']
+      options: [{noStrings: true}]
     }
   ],
 
@@ -240,15 +240,15 @@ ruleTester.run('jsx-no-literals', rule, {
         </Foo>
       `,
       parser: 'babel-eslint',
-      options: ['no-strings'],
+      options: [{noStrings: true}],
       errors: [{message: 'Strings not allowed in JSX files'}]
     }, {
       code: `
         <Foo bar="test">
           {'Test'}
         </Foo>
       `,
-      options: ['no-strings'],
+      options: [{noStrings: true}],
       errors: [{message: 'Strings not allowed in JSX files'}]
     }, {
       code: `
@@ -257,15 +257,15 @@ ruleTester.run('jsx-no-literals', rule, {
         </Foo>
       `,
       parser: 'babel-eslint',
-      options: ['no-strings'],
+      options: [{noStrings: true}],
       errors: [{message: 'Strings not allowed in JSX files'}]
     }, {
       code: `
         <Foo bar="test">
           Test
         </Foo>
       `,
-      options: ['no-strings'],
+      options: [{noStrings: true}],
       errors: [{message: 'Strings not allowed in JSX files'}]
     }
   ]