Skip to content

Add Vue support to 5 existing rules and enhance documentation#115

Open
neptia wants to merge 4 commits into
green-code-initiative:mainfrom
neptia:VUE-JS-RULES
Open

Add Vue support to 5 existing rules and enhance documentation#115
neptia wants to merge 4 commits into
green-code-initiative:mainfrom
neptia:VUE-JS-RULES

Conversation

@neptia

@neptia neptia commented May 19, 2026

Copy link
Copy Markdown

MODIFICATION OF EXISTING RULES

Added Vue SFC support for JSX‑based rules using Vue template AST visitors (vue-eslint-parser).
Updated RuleTester unit tests with Vue template cases for modified rules.
Added Vue integration examples in the test project and verified lint warnings on .vue files.
Covered rules: avoid-autoplay, no-empty-image-src-attribute, prefer-lighter-formats-for-image-files, avoid-css-animations, prefer-shorthand-css-notations.

@MP-Aubay MP-Aubay left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should also update CHANGELOG.md

Comment thread eslint-plugin/docs/rules/avoid-autoplay.md
Comment thread eslint-plugin/docs/rules/avoid-autoplay.md Outdated
Comment thread eslint-plugin/lib/rules/no-empty-image-src-attribute.js
Comment thread test-project/src/vue/avoid-autoplay.vue Outdated
Comment thread test-project/package.json Outdated
@MP-Aubay MP-Aubay moved this from Backlog to In Progress in Hackathon kanban - JS/TS May 20, 2026
@utarwyn

utarwyn commented May 30, 2026

Copy link
Copy Markdown
Member

Hello @neptia 👋 Thank you for the contribution!
Did you push the requested changes? I see you resolved them but no commit was added to the PR.

@utarwyn utarwyn requested a review from MP-Aubay May 31, 2026 19:23
Comment thread eslint-plugin/lib/rules/avoid-css-animations.js
Comment thread eslint-plugin/lib/rules/no-empty-image-src-attribute.js
Comment thread CHANGELOG.md Outdated
### Changed

- [#107](https://github.com/green-code-initiative/creedengo-javascript/pull/107) Move to creedengo-rules-specifications v3
- Add Vue SFC template support to JSX‑based rules (avoid-autoplay, no-empty-image-src-attribute, prefer-lighter-formats-for-image-files, avoid-css-animations, prefer-shorthand-css-notations)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you just add the PR URL before the description ?

@MP-Aubay

MP-Aubay commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

Hello @neptia, there is still 3 minors changes and I think it's good to go.

@utarwyn : The change resolved were correctly fixed

@utarwyn utarwyn requested a review from MP-Aubay June 8, 2026 18:38
@MP-Aubay

Copy link
Copy Markdown
Contributor

@utarwyn it's all good to me, do you want to check it before merge ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

Explore the integration of Vue.js rules

3 participants