Skip to content
This repository has been archived by the owner on Sep 27, 2023. It is now read-only.

Check only PHP files by default, include JS and CSS optionally #175

Open
dingo-d opened this issue Jul 29, 2019 · 0 comments
Open

Check only PHP files by default, include JS and CSS optionally #175

dingo-d opened this issue Jul 29, 2019 · 0 comments
Assignees
@dingo-d
Labels
Status: Future Release This issue will be fixed in the future release of the plugin, or an enhancement will be added in the Type: Enhancement This is an enhancement of the existing features or a new feature to the plugin.

Comments

@dingo-d
Copy link
Member

dingo-d commented Jul 29, 2019

Currently the sniff process can fail due to memory exhaustion when the sniffer comes across the minified JS file.

Currently, there is an option to check only PHP files. This option should work in reverse - if checked, JS files would be included in check (with a note that the sniff process may fail).

This would help with the current issues some authors and reviewers are experiencing.
@dingo-d dingo-d added Type: Enhancement This is an enhancement of the existing features or a new feature to the plugin. Status: Future Release This issue will be fixed in the future release of the plugin, or an enhancement will be added in the labels Jul 29, 2019
@dingo-d dingo-d self-assigned this Jul 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@dingo-d dingo-d

Labels
Status: Future Release This issue will be fixed in the future release of the plugin, or an enhancement will be added in the Type: Enhancement This is an enhancement of the existing features or a new feature to the plugin.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dingo-d