Skip to content

Commit 49c102b

Browse files
gsherwoodjrfnl
andcommitted
Remove all JS-specific sniffs
Removes the following sniffs: - Generic.Debug.ClosureLinter - Generic.Debug.JSHint - Generic.Debug.ESLint - Squiz.Classes.DuplicateProperty - Squiz.Debug.JavaScriptLint - Squiz.Debug.JSLint - Squiz.Objects.DisallowObjectStringIndex - Squiz.Objects.ObjectMemberComma - Squiz.WhiteSpace.PropertyLabelSpacing Includes removing two `<exclude name="Generic.Debug"/>` directives from tests for the Filter and Ruleset components. These excludes were only introduced temporarily to bypass the deprecation notice for sniffing JS/CSS files with the `Generic.Debug.*` sniffs. Ref: * squizlabs/PHP_CodeSniffer 2448 Co-authored-by: jrfnl <[email protected]>
1 parent 234c160 commit 49c102b

36 files changed

+0
-2169
lines changed

.cspell.json

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,7 @@
2828
"dnumber",
2929
"docblock",
3030
"encapsed",
31-
"gjslint",
3231
"groupid",
33-
"gsjlint",
3432
"hashbang",
3533
"heredoc",
3634
"hgblame",

.github/workflows/test.yml

Lines changed: 0 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -189,20 +189,6 @@ jobs:
189189
- name: "DEBUG: show libxml loaded version (php)"
190190
run: php -r 'echo "libxml loaded version = ", LIBXML_LOADED_VERSION, PHP_EOL;'
191191

192-
# This action also handles the caching of the dependencies.
193-
- name: Set up node
194-
if: ${{ matrix.custom_ini == false }}
195-
uses: actions/setup-node@v4
196-
with:
197-
node-version: '20'
198-
199-
- name: Install external tools used in tests
200-
if: ${{ matrix.custom_ini == false }}
201-
run: >
202-
npm install -g --fund false
203-
eslint
204-
jshint
205-
206192
# Install dependencies and handle caching in one go.
207193
# @link https://github.com/marketplace/actions/install-php-dependencies-with-composer
208194
- name: Install Composer dependencies
@@ -290,20 +276,6 @@ jobs:
290276
ini-values: error_reporting=-1, display_errors=On${{ steps.set_ini.outputs.PHP_INI }}
291277
coverage: xdebug
292278

293-
# This action also handles the caching of the dependencies.
294-
- name: Set up node
295-
if: ${{ matrix.custom_ini == false }}
296-
uses: actions/setup-node@v4
297-
with:
298-
node-version: '20'
299-
300-
- name: Install external tools used in tests
301-
if: ${{ matrix.custom_ini == false }}
302-
run: >
303-
npm install -g --fund false
304-
eslint
305-
jshint
306-
307279
# Install dependencies and handle caching in one go.
308280
# @link https://github.com/marketplace/actions/install-php-dependencies-with-composer
309281
- name: Install Composer dependencies

src/Standards/Generic/Docs/Debug/ClosureLinterStandard.xml

Lines changed: 0 additions & 19 deletions
This file was deleted.

src/Standards/Generic/Docs/Debug/JSHintStandard.xml

Lines changed: 0 additions & 19 deletions
This file was deleted.

src/Standards/Generic/Sniffs/Debug/ClosureLinterSniff.php

Lines changed: 0 additions & 156 deletions
This file was deleted.

0 commit comments

Comments
 (0)