Commit Graph

7 Commits

Author SHA1 Message Date
7c272b5399 t095: fix package.json lint coverage and quality gate (#103)
* fix: address PR #85 package.json review feedback (issue #95)

* fix: add trailing commas to cypress.config.js for comma-dangle rule

ESLint comma-dangle rule (always-multiline) requires trailing commas on
the last property and closing brace of multiline objects. Adding lint:js
coverage of cypress.config.js in package.json exposed these two missing
trailing commas at lines 15 and 16.

* fix: resolve ESLint errors blocking CI on PR #103

- Remove 'cypress/globals' env key from .eslintrc.js; this key requires
  eslint-plugin-cypress to be globally installed, which CI tools (Codacy,
  CodeFactor) do not have. Cypress globals are already declared explicitly
  in the globals block, making the env key redundant.
- Remove trailing commas from cypress.config.js to comply with the
  comma-dangle: never ESLint rule defined in .eslintrc.js.
- Update package-lock.json to include stylelint and stylelint-config-standard
  which were in devDependencies but missing from the lock file.
2026-03-18 21:33:54 +00:00
d6dcda908c fix: change comma-dangle rule to always-multiline per review feedback (#88)
Addresses Gemini review feedback from PR #66 (issue #73).
Changes 'comma-dangle' from 'never' to 'always-multiline' to improve
maintainability — cleaner git diffs and easier line reordering.
Updates admin/js files to comply with the new rule.

Closes #73
2026-03-17 21:00:19 +00:00
708acc39de fix: add comma-dangle rule to .eslintrc.js to prevent trailing commas (issue #25) (#66)
Add 'comma-dangle': ['error', 'never'] rule to .eslintrc.js to enforce
no trailing commas in JavaScript files. This addresses the Codacy review
findings from PR #15 that flagged trailing commas in object and array
literals as ErrorProne issues. The rule ensures ESLint catches trailing
commas automatically going forward.

Closes #25
2026-03-16 23:41:05 +00:00
40f6f596fa fix: resolve ESLint sourceType mismatch for cypress.config.js (issue #31) (#64)
Add overrides block in .eslintrc.js to parse cypress.config.js as CommonJS
(sourceType: 'script') rather than ESM. The global sourceType: 'module' caused
ESLint to flag require() as undefined, since require is not available in ESM
scope. The project has no 'type: module' in package.json, so CommonJS is correct.

Closes #31
2026-03-16 23:40:05 +00:00
0960368136 Fix ESLint global variable errors
Add WordPress global variables to ESLint config:
- jQuery (WordPress dependency)
- wpstData (localized script data)
- wpstModalData (modal script data)
- wp (WordPress JS API)

🤖 Generated with [Qoder][https://qoder.com]
2025-11-16 03:53:58 +00:00
e6dcda3f6e Fix GitHub Actions failures: code quality, tests, and linting
- Fix shellcheck warnings in bin/install-wp-tests.sh (quote variables, fix command -v usage)
- Remove trailing spaces in .github/workflows/phpunit.yml
- Add phpmd.xml to exclude camelCase checks for WordPress naming conventions
- Update composer.json to use phpmd.xml configuration
- Remove trailing commas in .eslintrc.js for Codacy compliance
- Add .markdownlint.json to configure markdown linting rules
- Improve Cypress test reliability with increased timeouts
- Update loginAsAdmin command with better error handling
- Make plugin activation checks more robust in Cypress tests

🤖 Generated with [Qoder][https://qoder.com]
2025-11-16 03:51:12 +00:00
b1966067ea Fix failing tests: Update install-wp-tests.sh, Cypress commands, and GitHub Actions workflows 2025-04-23 04:26:06 +01:00