fix: update dependencies

BREAKING CHANGE: potentially breaking dependency updates
gajus · Sep 14, 2021 · 00f0b4c · 00f0b4c
1 parent a76ebd8
commit 00f0b4c
Show file tree

Hide file tree

Showing 28 changed files with 53 additions and 53 deletions.
diff --git a/.README/rules/use-read-only-spread.md b/.README/rules/use-read-only-spread.md
@@ -18,8 +18,8 @@ You might expect the identifier name to be read-only, however, that's not true (
 ```flow js
 const x: Identifier = { name: '', type: '' };
 
-x.type = 'Should not be writable!'; // No Flow error
-x.name = 'Should not be writable!'; // No Flow error
+x.type = 'must NOT be writable!'; // No Flow error
+x.name = 'must NOT be writable!'; // No Flow error
 ```
 
 This rule suggests to use `$ReadOnly<…>` to prevent accidental loss of readonly-ness:
@@ -32,8 +32,8 @@ type Identifier = $ReadOnly<{|
 
 const x: Identifier = { name: '', type: '' };
 
-x.type = 'Should not be writable!'; // $FlowExpectedError[cannot-write]
-x.name = 'Should not be writable!'; // $FlowExpectedError[cannot-write]
+x.type = 'must NOT be writable!'; // $FlowExpectedError[cannot-write]
+x.name = 'must NOT be writable!'; // $FlowExpectedError[cannot-write]
 ```
 
 <!-- assertions useReadOnlySpread -->
diff --git a/README.md b/README.md
@@ -6625,8 +6625,8 @@ You might expect the identifier name to be read-only, however, that's not true (
 ```flow js
 const x: Identifier = { name: '', type: '' };
 
-x.type = 'Should not be writable!'; // No Flow error
-x.name = 'Should not be writable!'; // No Flow error
+x.type = 'must NOT be writable!'; // No Flow error
+x.name = 'must NOT be writable!'; // No Flow error
 ```
 
 This rule suggests to use `$ReadOnly<…>` to prevent accidental loss of readonly-ness:
@@ -6639,8 +6639,8 @@ type Identifier = $ReadOnly<{|
 
 const x: Identifier = { name: '', type: '' };
 
-x.type = 'Should not be writable!'; // $FlowExpectedError[cannot-write]
-x.name = 'Should not be writable!'; // $FlowExpectedError[cannot-write]
+x.type = 'must NOT be writable!'; // $FlowExpectedError[cannot-write]
+x.name = 'must NOT be writable!'; // $FlowExpectedError[cannot-write]
 ```
 
 The following patterns are considered problems:

diff --git a/src/rules/arrayStyle/index.js b/src/rules/arrayStyle/index.js
@@ -48,7 +48,7 @@ export default (defaultConfig, simpleType) => {
       GenericTypeAnnotation (node) {
         if (node.id.name === 'Array') {
           // Don't report on un-parameterized Array annotations. There are valid cases for this,
-          // but regardless, we should not crash when encountering them.
+          // but regardless, we must NOT crash when encountering them.
           if (node.typeParameters && node.typeParameters.params.length === 1) {
             const elementTypeNode = node.typeParameters.params[0];
             const rawElementType = context.getSourceCode().getText(elementTypeNode);

diff --git a/tests/rules/assertions/arrayStyleComplexType.js b/tests/rules/assertions/arrayStyleComplexType.js
@@ -64,7 +64,7 @@ export default {
       errors: [
         {
           data: 'normal',
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {

diff --git a/tests/rules/assertions/arrayStyleSimpleType.js b/tests/rules/assertions/arrayStyleSimpleType.js
@@ -78,7 +78,7 @@ export default {
       errors: [
         {
           data: 'normal',
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {

diff --git a/tests/rules/assertions/arrowParens.js b/tests/rules/assertions/arrowParens.js
@@ -209,7 +209,7 @@ export default {
   //     errors: [
   //       {
   //         keyword: 'enum',
-  //         dataPath: '[0]',
+  //         instancePath: '/0',
   //         schemaPath: '#/items/0/enum',
   //         params: {
   //           allowedValues: [

diff --git a/tests/rules/assertions/booleanStyle.js b/tests/rules/assertions/booleanStyle.js
@@ -23,7 +23,7 @@ export default {
       errors: [
         {
           data: 'integer',
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {

diff --git a/tests/rules/assertions/delimiterDangle.js b/tests/rules/assertions/delimiterDangle.js
@@ -946,7 +946,7 @@ const MISCONFIGURED = [
     errors: [
       {
         data: 'occasionally',
-        dataPath: '[0]',
+        instancePath: '/0',
         keyword: 'enum',
         message: 'must be equal to one of the allowed values',
         params: {

diff --git a/tests/rules/assertions/genericSpacing.js b/tests/rules/assertions/genericSpacing.js
@@ -97,7 +97,7 @@ export default {
       errors: [
         {
           data: 'frequently',
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {

diff --git a/tests/rules/assertions/interfaceIdMatch.js b/tests/rules/assertions/interfaceIdMatch.js
@@ -25,7 +25,7 @@ export default {
       errors: [
         {
           data: 7,
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'type',
           message: 'must be string',
           params: {

diff --git a/tests/rules/assertions/noFlowFixMeComments.js b/tests/rules/assertions/noFlowFixMeComments.js
@@ -55,7 +55,7 @@ export default {
       errors: [
         {
           data: 7,
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'type',
           message: 'must be string',
           params: {

diff --git a/tests/rules/assertions/noWeakTypes.js b/tests/rules/assertions/noWeakTypes.js
@@ -200,9 +200,9 @@ export default {
           data: {
             nonExistentWeakType: false,
           },
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'additionalProperties',
-          message: 'should NOT have additional properties',
+          message: 'must NOT have additional properties',
           params: {
             additionalProperty: 'nonExistentWeakType',
           },
@@ -231,7 +231,7 @@ export default {
       errors: [
         {
           data: 'irrelevant',
-          dataPath: '[0].Object',
+          instancePath: '/0/Object',
           keyword: 'type',
           message: 'must be boolean',
           params: {

diff --git a/tests/rules/assertions/objectTypeCurlySpacing.js b/tests/rules/assertions/objectTypeCurlySpacing.js
@@ -118,7 +118,7 @@ export default {
       errors: [
         {
           data: 'sometimes',
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {

diff --git a/tests/rules/assertions/objectTypeDelimiter.js b/tests/rules/assertions/objectTypeDelimiter.js
@@ -90,7 +90,7 @@ export default {
       errors: [
         {
           data: 'period',
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {

diff --git a/tests/rules/assertions/quotes.js b/tests/rules/assertions/quotes.js
@@ -74,7 +74,7 @@ export default {
       errors: [
         {
           data: 'temporarily',
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {

diff --git a/tests/rules/assertions/requireCompoundTypeAlias.js b/tests/rules/assertions/requireCompoundTypeAlias.js
@@ -48,7 +48,7 @@ export default {
       errors: [
         {
           data: 'sometimes',
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {

diff --git a/tests/rules/assertions/requireParameterType.js b/tests/rules/assertions/requireParameterType.js
@@ -142,9 +142,9 @@ export default {
           data: {
             excludeOtherStuff: true,
           },
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'additionalProperties',
-          message: 'should NOT have additional properties',
+          message: 'must NOT have additional properties',
           params: {
             additionalProperty: 'excludeOtherStuff',
           },
@@ -174,7 +174,7 @@ export default {
       errors: [
         {
           data: 'everything',
-          dataPath: '[0].excludeArrowFunctions',
+          instancePath: '/0/excludeArrowFunctions',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {
@@ -205,7 +205,7 @@ export default {
       errors: [
         {
           data: 3,
-          dataPath: '[0].excludeParameterMatch',
+          instancePath: '/0/excludeParameterMatch',
           keyword: 'type',
           message: 'must be string',
           params: {

diff --git a/tests/rules/assertions/requireReturnType.js b/tests/rules/assertions/requireReturnType.js
@@ -441,9 +441,9 @@ export default {
           data: {
             excludeOtherStuff: true,
           },
-          dataPath: '[1]',
+          instancePath: '/1',
           keyword: 'additionalProperties',
-          message: 'should NOT have additional properties',
+          message: 'must NOT have additional properties',
           params: {
             additionalProperty: 'excludeOtherStuff',
           },
@@ -491,7 +491,7 @@ export default {
       errors: [
         {
           data: 'often',
-          dataPath: '[1].annotateUndefined',
+          instancePath: '/1/annotateUndefined',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {
@@ -526,7 +526,7 @@ export default {
       errors: [
         {
           data: 'everything',
-          dataPath: '[1].excludeArrowFunctions',
+          instancePath: '/1/excludeArrowFunctions',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {
@@ -557,7 +557,7 @@ export default {
       errors: [
         {
           data: '^foo',
-          dataPath: '[1].excludeMatching',
+          instancePath: '/1/excludeMatching',
           keyword: 'type',
           message: 'must be array',
           params: {
@@ -579,7 +579,7 @@ export default {
       errors: [
         {
           data: false,
-          dataPath: '[1].includeOnlyMatching[0]',
+          instancePath: '/1/includeOnlyMatching/0',
           keyword: 'type',
           message: 'must be string',
           params: {

diff --git a/tests/rules/assertions/requireTypesAtTop.js b/tests/rules/assertions/requireTypesAtTop.js
@@ -30,7 +30,7 @@ export default {
       errors: [
         {
           data: 'sometimes',
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {

diff --git a/tests/rules/assertions/requireValidFileAnnotation.js b/tests/rules/assertions/requireValidFileAnnotation.js
@@ -278,7 +278,7 @@ export default {
       errors: [
         {
           data: 'sometimes',
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {
@@ -307,7 +307,7 @@ export default {
       errors: [
         {
           data: 'upside-down',
-          dataPath: '[1].annotationStyle',
+          instancePath: '/1/annotationStyle',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {

diff --git a/tests/rules/assertions/requireVariableType.js b/tests/rules/assertions/requireVariableType.js
@@ -53,9 +53,9 @@ export default {
           data: {
             excludeOtherStuff: true,
           },
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'additionalProperties',
-          message: 'should NOT have additional properties',
+          message: 'must NOT have additional properties',
           params: {
             additionalProperty: 'excludeOtherStuff',
           },
@@ -93,7 +93,7 @@ export default {
       errors: [
         {
           data: 99,
-          dataPath: '[0].excludeVariableMatch',
+          instancePath: '/0/excludeVariableMatch',
           keyword: 'type',
           message: 'must be string',
           params: {
@@ -114,9 +114,9 @@ export default {
           data: {
             declare: false,
           },
-          dataPath: '[0].excludeVariableTypes',
+          instancePath: '/0/excludeVariableTypes',
           keyword: 'additionalProperties',
-          message: 'should NOT have additional properties',
+          message: 'must NOT have additional properties',
           params: {
             additionalProperty: 'declare',
           },
@@ -145,7 +145,7 @@ export default {
       errors: [
         {
           data: 'yes',
-          dataPath: '[0].excludeVariableTypes.let',
+          instancePath: '/0/excludeVariableTypes/let',
           keyword: 'type',
           message: 'must be boolean',
           params: {

diff --git a/tests/rules/assertions/semi.js b/tests/rules/assertions/semi.js
@@ -66,7 +66,7 @@ export default {
       errors: [
         {
           data: 'temporarily',
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {

diff --git a/tests/rules/assertions/sortKeys.js b/tests/rules/assertions/sortKeys.js
@@ -628,7 +628,7 @@ export default {
       errors: [
         {
           data: 'random',
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {

diff --git a/tests/rules/assertions/spaceAfterTypeColon.js b/tests/rules/assertions/spaceAfterTypeColon.js
@@ -1120,7 +1120,7 @@ const MISCONFIGURED = [
     errors: [
       {
         data: 'from time to time',
-        dataPath: '[0]',
+        instancePath: '/0',
         keyword: 'enum',
         message: 'must be equal to one of the allowed values',
         params: {
@@ -1151,9 +1151,9 @@ const MISCONFIGURED = [
         data: {
           allowEmoji: true,
         },
-        dataPath: '[1]',
+        instancePath: '/1',
         keyword: 'additionalProperties',
-        message: 'should NOT have additional properties',
+        message: 'must NOT have additional properties',
         params: {
           additionalProperty: 'allowEmoji',
         },
@@ -1176,7 +1176,7 @@ const MISCONFIGURED = [
     errors: [
       {
         data: 'why not?',
-        dataPath: '[1].allowLineBreak',
+        instancePath: '/1/allowLineBreak',
         keyword: 'type',
         message: 'must be boolean',
         params: {

diff --git a/tests/rules/assertions/spaceBeforeGenericBracket.js b/tests/rules/assertions/spaceBeforeGenericBracket.js
@@ -34,7 +34,7 @@ export default {
       errors: [
         {
           data: 'whenever',
-          dataPath: '[0]',
+          instancePath: '/0',
           keyword: 'enum',
           message: 'must be equal to one of the allowed values',
           params: {

diff --git a/tests/rules/assertions/spaceBeforeTypeColon.js b/tests/rules/assertions/spaceBeforeTypeColon.js
@@ -927,7 +927,7 @@ const MISCONFIGURED = [
     errors: [
       {
         data: 'wherever',
-        dataPath: '[0]',
+        instancePath: '/0',
         keyword: 'enum',
         message: 'must be equal to one of the allowed values',
         params: {