Skip to content

Commit

Permalink
Merge pull request #1338 from Automattic/update/eslint-wpvip
Browse files Browse the repository at this point in the history
chore(dev-deps): Update @automattic/eslint-plugin-wpvip from 0.4.0 to 0.5.2
  • Loading branch information
sjinks authored Apr 20, 2023
2 parents 1784136 + 43c2b5d commit fee5186
Show file tree
Hide file tree
Showing 17 changed files with 775 additions and 1,443 deletions.
3 changes: 0 additions & 3 deletions .eslintrc.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,6 @@
"rules": {
"no-console": 0,
"no-process-exit": 0,
"jsdoc/check-tag-names": [1, {
"definedTags": ["flow", "format"]
}],
"security/detect-object-injection": 0,
"security/detect-non-literal-fs-filename": 0,
"security/detect-unsafe-regex": 0,
Expand Down
3 changes: 2 additions & 1 deletion __tests__/bin/vip-config-envvar-delete.js
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,8 @@ const mockDeleteEnvVar: JestMockFn<[number, number, string], Promise<void>> = de
const mockTrackEvent: JestMockFn<[], Promise<Response>> = trackEvent;

describe( 'deleteEnvVarCommand', () => {
let args, opts;
let args;
let opts;
const eventPayload = expect.objectContaining( { command: expect.stringContaining( 'vip @mysite.develop config envvar delete' ) } );
const executeEvent = [ 'envvar_delete_command_execute', eventPayload ];
const successEvent = [ 'envvar_delete_command_success', eventPayload ];
Expand Down
3 changes: 2 additions & 1 deletion __tests__/bin/vip-config-envvar-set.js
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,8 @@ describe( 'vip config envvar set', () => {
} );

describe( 'setEnvVarCommand', () => {
let args, opts;
let args;
let opts;
const eventPayload = expect.objectContaining( { command: expect.stringContaining( 'vip @mysite.develop config envvar set' ) } );
const executeEvent = [ 'envvar_set_command_execute', eventPayload ];
const successEvent = [ 'envvar_set_command_success', eventPayload ];
Expand Down
2 changes: 1 addition & 1 deletion __tests__/commands/dev-env-sync-sql.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
import { describe, expect, jest } from '@jest/globals';
import fs from 'fs';
import { replace } from '@automattic/vip-search-replace';
import { PassThrough, Stream } from 'stream';
import { PassThrough } from 'stream';

/**
* Internal dependencies
Expand Down
6 changes: 3 additions & 3 deletions __tests__/devenv-e2e/helpers/cli-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,9 @@ export class CliTest {
spawn( args, options, printStderrOnError ) {
const [ command, ...commandArgs ] = args;

let stdout = '',
stderr = '',
finished = false;
let stdout = '';
let stderr = '';
let finished = false;

return new Promise( ( resolve, reject ) => {
const child = spawn( command, commandArgs, options );
Expand Down
1 change: 0 additions & 1 deletion __tests__/devenv-e2e/helpers/docker-utils.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
/* eslint-disable jsdoc/valid-types */
/* eslint-disable id-length */

/**
Expand Down
2 changes: 0 additions & 2 deletions __tests__/devenv-e2e/helpers/utils.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
/* eslint-disable jsdoc/valid-types */

/**
* External dependencies
*/
Expand Down
1 change: 0 additions & 1 deletion __tests__/devenv-e2e/jest/jest.config.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
/* eslint-disable jsdoc/valid-types */
const path = require( 'node:path' );

/** @type {import('jest').Config} */
Expand Down
1 change: 0 additions & 1 deletion __tests__/devenv-e2e/jest/sequencer.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
/* eslint-disable jsdoc/valid-types */
const Sequencer = require( '@jest/test-sequencer' ).default;

/**
Expand Down
Loading

0 comments on commit fee5186

Please sign in to comment.