[#47962] - [6.2] eslint config (stylistic)
- Fixed in Code Base
- 6 Jul 2026
- Medium
- Build: 6.2-dev
- # 47962
- Diff
- heelc29:6.2/issue-45891-eslint
User tests: Successful: Unsuccessful:
Pull Request resolves #45891, #47199 and #46877 (comment).
Same as #47272 and #47273 but for 6.2 as new dev dependencies
- I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.
Summary of Changes
- add
@stylistic/eslint-pluginto get basic code style checks for javascript files back - add
eslint-plugin-cypressto check files for system tests (cypress) and fix #47199 - update eslint config to use
eslint-plugin-vue - reapply #44852 to exit on warnings
- fix existing code style errors
Testing Instructions
run npm run lint:js and npm run lint:testjs
Actual result BEFORE applying this Pull Request
basic code style (indentation, comma, semicolon) is not checked
vue files are not checked
Expected result AFTER applying this Pull Request
basic code style (indentation, comma, semicolon) is checked (again)
vue files are checked
warnings results in exit code 1
Link to documentations
Please select:
- No documentation changes for guide.joomla.org needed
- No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | JavaScript Repository |
![avatar github-actions[bot]](/images/avatars/github-actions[bot].png)
| Labels |
Added:
Conflicting Files
PR-6.2-dev
|
||
This pull request has conflicts, please resolve those before we can evaluate the pull request.
I have tested this item ✅ successfully on 00a8b5f
Tested with JBT
- Before PR all three style hacks (indent, comma, semicolon) are not reported on:
- one JS file in media/
- one JS file in tests/
- Applied PR with Patch Tester and
npm ci - After PR the modifications are shown for files with
lint:jsandlint:testjs- error Missing semicolon
- error Expected indentation of 2 spaces but found 0
- error Unexpected trailing comma
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47962.
I have tested this item ✅ successfully on 00a8b5f
Tested with JBT
- Before PR all three style hacks (indent, comma, semicolon) are not reported on:
- one JS file in media/
- one JS file in tests/
- Applied PR with Patch Tester and
npm ci - After PR the modifications are shown for files with
lint:jsandlint:testjs- error Missing semicolon
- error Expected indentation of 2 spaces but found 0
- error Unexpected trailing comma
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47962.
I have tested this item ✅ successfully on 00a8b5f
Tested
- change of ',' in a function call
- double ';;' at the end of a line
- changed indention of lines
on files:
- \tests\System\integration\install\Installation.cy.js
- \media_source\com_modules\js\admin-modules-modal.es6.js
Changes were not remarked before applying and were marked afterwards
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47962.
I have tested this item ✅ successfully on 00a8b5f
Tested
- change of ',' in a function call
- double ';;' at the end of a line
- changed indention of lines
on files:
- \tests\System\integration\install\Installation.cy.js
- \media_source\com_modules\js\admin-modules-modal.es6.js
Changes were not remarked before applying and were marked afterwards
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47962.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47962.
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47962.
| Labels |
Added:
RTC
Removed: Conflicting Files |
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2026-07-06 15:18:35 |
| Closed_By | ⇒ | MacJoom |
Thank you!
This pull request has conflicts, please resolve those before we can evaluate the pull request.