nebius/SWE-rebench-V2-PRs huggingface dataset:
Repository: test-results-reporter/reporter
Pull request #3 (2022-03-02 09:38:04)

Update vertical divider


Patch:
diff --git a/package.json b/package.json
index b5540ac..8a7a52a 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "test-results-reporter",
-  "version": "0.0.19",
+  "version": "0.0.20",
   "description": "Publish test results to Microsoft Teams and Slack",
   "main": "src/index.js",
   "bin": {
diff --git a/src/targets/slack.js b/src/targets/slack.js
index 81fe1c7..29a39e1 100644
--- a/src/targets/slack.js
+++ b/src/targets/slack.js
@@ -70,7 +70,7 @@ async function getReportPortalAnalysisAttachment(opts) {
         "fields": [
           {
             "title": "Report Portal Analysis",
-            "value": results.join(' | '),
+            "value": results.join(' ｜ '),
             "short": false
           }
         ]
@@ -89,7 +89,7 @@ function getLinks(options) {
   }
   return {
     "fallback": "links",
-    "footer": links.join(' | ')
+    "footer": links.join(' ｜ ')
   }
 }
 
@@ -272,4 +272,4 @@ async function send(options, results) {
 
 module.exports = {
   send
-}
\ No newline at end of file
+}
diff --git a/src/targets/teams.js b/src/targets/teams.js
index 8eb73b7..613e783 100644
--- a/src/targets/teams.js
+++ b/src/targets/teams.js
@@ -89,7 +89,7 @@ async function getReportPortalAnalysisBlocks(opts) {
         },
         {
           "type": "TextBlock",
-          "text": results.join(' | ')
+          "text": results.join(' ｜ ')
         }
       ]
     }
@@ -106,7 +106,7 @@ function getLinks(opts) {
   }
   return {
     "type": "TextBlock",
-    "text": links.join(' | '),
+    "text": links.join(' ｜ '),
     "separator": true
   }
 }
@@ -362,4 +362,4 @@ async function send(options, results) {
 
 module.exports = {
   send
-}
\ No newline at end of file
+}


Test patch:
diff --git a/test/helpers/interactions.js b/test/helpers/interactions.js
index 9a6652d..c1e97ae 100644
--- a/test/helpers/interactions.js
+++ b/test/helpers/interactions.js
@@ -1183,7 +1183,7 @@ addInteractionHandler('post test-summary to teams with report portal analysis',
                 },
                 {
                   "type": "TextBlock",
-                  "text": "🔴 PB - 0 | 🟡 AB - 0 | 🔵 SI - 0 | ◯ ND - 0 | **🟠 TI - 4**"
+                  "text": "🔴 PB - 0 ｜ 🟡 AB - 0 ｜ 🔵 SI - 0 ｜ ◯ ND - 0 ｜ **🟠 TI - 4**"
                 },
                 {
                   "type": "TextBlock",
@@ -1233,7 +1233,7 @@ addInteractionHandler('post test-summary to slack with report portal analysis',
             "fields": [
               {
                 "title": "Report Portal Analysis",
-                "value": "🔴 PB - 0 | 🟡 AB - 0 | 🔵 SI - 0 | ◯ ND - 0 | *🟠 TI - 4*",
+                "value": "🔴 PB - 0 ｜ 🟡 AB - 0 ｜ 🔵 SI - 0 ｜ ◯ ND - 0 ｜ *🟠 TI - 4*",
                 "short": false
               }
             ]
@@ -1250,4 +1250,4 @@ addInteractionHandler('post test-summary to slack with report portal analysis',
       status: 200
     }
   }
-});
\ No newline at end of file
+});
Repository: vuejs/eslint-plugin-vue
Pull request #33 (2017-06-17 18:06:56)

Fix Should typo
There's a small typo on `html-no-self-closing` rule, `shuld` => `should`


Patch:
diff --git a/lib/rules/html-no-self-closing.js b/lib/rules/html-no-self-closing.js
index 19848cb9..0a9a3678 100644
--- a/lib/rules/html-no-self-closing.js
+++ b/lib/rules/html-no-self-closing.js
@@ -29,7 +29,7 @@ function create (context) {
         context.report({
           node,
           loc: node.loc,
-          message: 'Self-closing shuld not be used.',
+          message: 'Self-closing should not be used.',
           fix: (fixer) => fixer.removeRange([pos, pos + 1])
         })
       }


Test patch:
diff --git a/tests/lib/rules/html-no-self-closing.js b/tests/lib/rules/html-no-self-closing.js
index 3f298fb4..26f8edb8 100644
--- a/tests/lib/rules/html-no-self-closing.js
+++ b/tests/lib/rules/html-no-self-closing.js
@@ -45,19 +45,19 @@ tester.run('html-no-self-closing', rule, {
       filename: 'test.vue',
       code: '<template><div><br/></div></template>',
       output: '<template><div><br></div></template>',
-      errors: ['Self-closing shuld not be used.']
+      errors: ['Self-closing should not be used.']
     },
     {
       filename: 'test.vue',
       code: '<template><div><input/></div></template>',
       output: '<template><div><input></div></template>',
-      errors: ['Self-closing shuld not be used.']
+      errors: ['Self-closing should not be used.']
     },
     {
       filename: 'test.vue',
       code: '<template><div><div/></div></template>',
       output: '<template><div><div></div></template>',
-      errors: ['Self-closing shuld not be used.']
+      errors: ['Self-closing should not be used.']
     }
   ]
 })
Repository: vuejs/eslint-plugin-vue
Pull request #38 (2017-06-20 02:52:23)

Chore: add issue template
I created the issue template based on [ESLint's issue template](https://github.com/eslint/eslint/blob/master/.github/ISSUE_TEMPLATE.md).
I expect that the issue template helps us to get information what we need to investigate bugs.


Patch:
diff --git a/.github/ISSUE_TEMPLATE.md b/.github/ISSUE_TEMPLATE.md
new file mode 100644
index 00000000..9a3407f1
--- /dev/null
+++ b/.github/ISSUE_TEMPLATE.md
@@ -0,0 +1,39 @@
+<!--
+    This template is for bug reports. If you are reporting a bug, please continue on. If you are here for another reason, please see below:
+
+    1. To propose a new rule: http://eslint.org/docs/developer-guide/contributing/new-rules
+    2. To request a change: http://eslint.org/docs/developer-guide/contributing/changes
+
+    Note that leaving sections blank will make it difficult for us to troubleshoot and we may have to close the issue.
+-->
+
+**Tell us about your environment**
+
+* **ESLint Version:**
+* **eslint-plugin-vue Version:**
+* **Node Version:**
+
+**Please show your full configuration:**
+
+<!-- Paste your configuration below: -->
+```
+
+
+
+
+```
+
+**What did you do? Please include the actual source code causing the issue.**
+
+<!-- Paste the source code below: -->
+```vue
+
+
+
+
+
+```
+
+**What did you expect to happen?**
+
+**What actually happened? Please include the actual, raw output from ESLint.**
diff --git a/lib/rules/no-invalid-v-for.js b/lib/rules/no-invalid-v-for.js
index b9852b57..9b19599a 100644
--- a/lib/rules/no-invalid-v-for.js
+++ b/lib/rules/no-invalid-v-for.js
@@ -21,6 +21,9 @@ const utils = require('../utils')
  * @returns {boolean} `true` if the node is using the variables which are defined by `v-for` directives.
  */
 function isUsingIterationVar (node) {
+  if (node.value == null) {
+    return false
+  }
   const references = node.value.references
   const variables = node.parent.parent.variables
 
diff --git a/package.json b/package.json
index 37602fdd..d58481b7 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-vue",
-  "version": "3.1.1",
+  "version": "3.1.2",
   "description": "Official ESLint plugin for Vue.js",
   "main": "lib/index.js",
   "scripts": {


Test patch:
diff --git a/tests/lib/rules/no-invalid-v-for.js b/tests/lib/rules/no-invalid-v-for.js
index 86119773..f465b567 100644
--- a/tests/lib/rules/no-invalid-v-for.js
+++ b/tests/lib/rules/no-invalid-v-for.js
@@ -144,6 +144,11 @@ tester.run('no-invalid-v-for', rule, {
       filename: 'test.vue',
       code: '<template><div><custom-component v-for="x in list" :key="foo"></custom-component></div></template>',
       errors: ["Expected 'v-bind:key' directive to use the variables which are defined by the 'v-for' directive."]
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div><div v-for="(item, index) in suggestions" :key></div></div></template>',
+      errors: ["Expected 'v-bind:key' directive to use the variables which are defined by the 'v-for' directive."]
     }
   ]
 })
Repository: vuejs/eslint-plugin-vue
Pull request #143 (2017-08-09 12:43:47)

Fix small mistake in `require-valid-default-prop`
- Improve coverage for FunctionExpression and ArrowFunctionExpression
- Fix traversing over many types (return -> continue)

armano2: @michalsnik i tested this rule in few big projects and i found few mistakes what i made there,

- If there is more than one prop, traversing over props is stopped if type is valid
- Function and variables are throwing errors

Patch:
diff --git a/lib/rules/require-valid-default-prop.js b/lib/rules/require-valid-default-prop.js
index 22dbe5b4..659df018 100644
--- a/lib/rules/require-valid-default-prop.js
+++ b/lib/rules/require-valid-default-prop.js
@@ -102,23 +102,20 @@ module.exports = {
 
       for (const prop of properties) {
         const type = getPropertyNode(prop.value, 'type')
-        if (!type) {
-          return
-        }
+        if (!type) continue
 
         const typeNames = new Set(getTypes(type.value)
           .map(item => item === 'Object' || item === 'Array' ? 'Function' : item) // Object and Array require function
           .filter(item => NATIVE_TYPES.has(item)))
 
-        if (typeNames.size === 0) { // There is no native types detected
-          return
-        }
+        // There is no native types detected
+        if (typeNames.size === 0) continue
 
         const def = getPropertyNode(prop.value, 'default')
-        if (!def) return
+        if (!def) continue
 
         const defType = getValueType(def.value)
-        if (typeNames.has(defType)) return
+        if (!defType || typeNames.has(defType)) continue
 
         context.report({
           node: def,


Test patch:
diff --git a/tests/lib/rules/require-valid-default-prop.js b/tests/lib/rules/require-valid-default-prop.js
index 8c4a84ab..b95fa71a 100644
--- a/tests/lib/rules/require-valid-default-prop.js
+++ b/tests/lib/rules/require-valid-default-prop.js
@@ -65,6 +65,7 @@ ruleTester.run('require-valid-default-prop', rule, {
           foo: null,
           foo: Number,
           foo: [String, Number],
+          foo: { type: Number, default: VAR_BAR },
           foo: { type: Number, default: 100 },
           foo: { type: [String, Number], default: '' },
           foo: { type: [String, Number], default: 0 },
Repository: vuejs/eslint-plugin-vue
Pull request #331 (2018-01-07 07:37:22)

Fix: unintentional disabling
`<!-- eslint-disable-line foo -->` was parsed to `<!-- eslint-disable -line foo -->` (i.e. disable 2 rules `-line` and `foo` after the comment) wrongly. This PR fixes the bug -- it requires one or more spaces between `eslint-disable` and the first rule name.

This is a bug fix which increases warnings, so needs `semver-minor`.
  


Patch:
diff --git a/lib/rules/comment-directive.js b/lib/rules/comment-directive.js
index 4afe8109..1e53749d 100644
--- a/lib/rules/comment-directive.js
+++ b/lib/rules/comment-directive.js
@@ -8,8 +8,8 @@
 // Helpers
 // -----------------------------------------------------------------------------
 
-const COMMENT_DIRECTIVE_B = /^\s*(eslint-(?:en|dis)able)\s*(?:(\S|\S[\s\S]*\S)\s*)?$/
-const COMMENT_DIRECTIVE_L = /^\s*(eslint-disable(?:-next)?-line)\s*(?:(\S|\S[\s\S]*\S)\s*)?$/
+const COMMENT_DIRECTIVE_B = /^\s*(eslint-(?:en|dis)able)(?:\s+(\S|\S[\s\S]*\S))?\s*$/
+const COMMENT_DIRECTIVE_L = /^\s*(eslint-disable(?:-next)?-line)(?:\s+(\S|\S[\s\S]*\S))?\s*$/
 
 /**
  * Parse a given comment.


Test patch:
diff --git a/tests/lib/rules/comment-directive.js b/tests/lib/rules/comment-directive.js
index 32de76a6..367d29dd 100644
--- a/tests/lib/rules/comment-directive.js
+++ b/tests/lib/rules/comment-directive.js
@@ -191,5 +191,22 @@ describe('comment-directive', () => {
       assert.deepEqual(messages[0].ruleId, 'vue/no-parsing-error')
       assert.deepEqual(messages[1].ruleId, 'vue/no-duplicate-attributes')
     })
+
+    it('should affect only the next line', () => {
+      const code = `
+        <template>
+          <!-- eslint-disable-next-line vue/no-parsing-error, vue/no-duplicate-attributes -->
+          <div id id="a">Hello</div>
+          <div id id="b">Hello</div>
+        </template>
+      `
+      const messages = linter.executeOnText(code, 'test.vue').results[0].messages
+
+      assert.deepEqual(messages.length, 2)
+      assert.deepEqual(messages[0].ruleId, 'vue/no-parsing-error')
+      assert.deepEqual(messages[0].line, 5)
+      assert.deepEqual(messages[1].ruleId, 'vue/no-duplicate-attributes')
+      assert.deepEqual(messages[1].line, 5)
+    })
   })
 })