Skip to content

Commit 8c83767

Browse files
Merge pull request #192 from MikeMcC399/update/meta-description
chore: standardize meta.docs.description property
2 parents bd9694f + 9d51b31 commit 8c83767

10 files changed

+10
-9
lines changed

.eslintrc.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
"pattern": "https://github.com/cypress-io/eslint-plugin-cypress/blob/master/docs/rules/{{name}}.md"
2222
}
2323
],
24+
"eslint-plugin/require-meta-docs-description": "error",
2425
"n/no-extraneous-require": [
2526
"error",
2627
{

lib/rules/assertion-before-screenshot.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ module.exports = {
1818
meta: {
1919
type: 'problem',
2020
docs: {
21-
description: 'Assert on the page state before taking a screenshot, so the screenshot is consistent',
21+
description: 'require screenshots to be preceded by an assertion',
2222
category: 'Possible Errors',
2323
recommended: false,
2424
url: 'https://github.com/cypress-io/eslint-plugin-cypress/blob/master/docs/rules/assertion-before-screenshot.md',

lib/rules/no-assigning-return-values.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ module.exports = {
1919
meta: {
2020
type: 'problem',
2121
docs: {
22-
description: 'Prevent assigning return values of cy calls',
22+
description: 'disallow assigning return values of `cy` calls',
2323
category: 'Possible Errors',
2424
recommended: true,
2525
url: 'https://github.com/cypress-io/eslint-plugin-cypress/blob/master/docs/rules/no-assigning-return-values.md',

lib/rules/no-async-before.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ module.exports = {
44
meta: {
55
type: 'problem',
66
docs: {
7-
description: 'Prevent using async/await in Cypress before methods',
7+
description: 'disallow using `async`/`await` in Cypress `before` methods',
88
category: 'Possible Errors',
99
recommended: true,
1010
url: 'https://github.com/cypress-io/eslint-plugin-cypress/blob/master/docs/rules/no-async-before.md',

lib/rules/no-async-tests.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ module.exports = {
44
meta: {
55
type: 'problem',
66
docs: {
7-
description: 'Prevent using async/await in Cypress test cases',
7+
description: 'disallow using `async`/`await` in Cypress test cases',
88
category: 'Possible Errors',
99
recommended: true,
1010
url: 'https://github.com/cypress-io/eslint-plugin-cypress/blob/master/docs/rules/no-async-tests.md',

lib/rules/no-force.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ module.exports = {
88
meta: {
99
type: 'suggestion',
1010
docs: {
11-
description: 'Disallow using of \'force: true\' option for click and type calls',
11+
description: 'disallow using `force: true` with action commands',
1212
category: 'Possible Errors',
1313
recommended: false,
1414
url: 'https://github.com/cypress-io/eslint-plugin-cypress/blob/master/docs/rules/no-force.md',

lib/rules/no-pause.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ module.exports = {
88
meta: {
99
type: 'suggestion',
1010
docs: {
11-
description: 'Disallow using of \'cy.pause\' calls',
11+
description: 'disallow using `cy.pause()` calls',
1212
category: 'Possible Errors',
1313
recommended: false,
1414
url: 'https://github.com/cypress-io/eslint-plugin-cypress/blob/master/docs/rules/no-pause.md',

lib/rules/no-unnecessary-waiting.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ module.exports = {
44
meta: {
55
type: 'problem',
66
docs: {
7-
description: 'Prevent waiting for arbitrary time periods',
7+
description: 'disallow waiting for arbitrary time periods',
88
category: 'Possible Errors',
99
recommended: true,
1010
url: 'https://github.com/cypress-io/eslint-plugin-cypress/blob/master/docs/rules/no-unnecessary-waiting.md',

lib/rules/require-data-selectors.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ module.exports = {
44
meta: {
55
type: 'suggestion',
66
docs: {
7-
description: 'Use data-* attributes to provide context to your selectors and insulate them from CSS or JS changes',
7+
description: 'require `data-*` attribute selectors',
88
category: 'Possible Errors',
99
recommended: false,
1010
url: 'https://github.com/cypress-io/eslint-plugin-cypress/blob/master/docs/rules/require-data-selectors.md',

lib/rules/unsafe-to-chain-command.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
const { basename } = require('path')
44

55
const NAME = basename(__dirname)
6-
const DESCRIPTION = 'Actions should be in the end of chains, not in the middle'
6+
const DESCRIPTION = 'disallow actions within chains'
77

88
/**
99
* Commands listed in the documentation with text: 'It is unsafe to chain further commands that rely on the subject after xxx.'

0 commit comments

Comments
 (0)