Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature Request] Remove unclear PHP line comment #929

Open
tyrann0us opened this issue Nov 22, 2024 · 0 comments · May be fixed by #930
Open

[Feature Request] Remove unclear PHP line comment #929

tyrann0us opened this issue Nov 22, 2024 · 0 comments · May be fixed by #930

Comments

@tyrann0us
Copy link

Is your feature request related to a problem? Please describe.

// Σ(゚Д゚).

This unusual comment consisting of an emoji-like sequence (that possibly expresses a shocked or surprised face, according to ChatGPT) is triggering false positives in security scans of a hosting provider (https://maxcluster.de/en/knowledge-base/shopsecurity/what-is-shopsecurity). See the following screenshot:
Image

Describe the solution you'd like
It would be good if the comment could be removed to avoid further false positives and because its existence is unclear.

Describe alternatives you've considered
Alternatively, the comment should be converted into an explanatory sentence.

Additional context
It was introduced in https://github.com/wp-media/imagify-plugin/pull/435/files#diff-3b8e1c62f65c1f8867b62aed7de21bbdc91c753291e1d3dac8735d252a0ff3f7R220 by @Screenfeed; maybe he can say more about it.

@tyrann0us tyrann0us linked a pull request Nov 22, 2024 that will close this issue
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant