Replies: 0
The wfi18n.js script moreless consequently skips semicolons to end variable assignments and other statements, where it is optional. While this is not wrong, it breaks the ability to minimise the script, as removed linebreaks merge two statements if no semicolon clearly separates them.
Since in all other Wordfence scripts, those semicolons are present, it seems to be no agreed coding standard on this project and I hence suggest to add the semicolons where required to keep syntax intact when linebreaks are removed as minimise step.
I can commit the change directly, but I’m new to this platform and not sure how to fork, commit or propose a merge/pull request. Feel free to guide me in this regards, so I can offer future contributions more effectively.
Kind regards,
Micha