Modifies i18n_usage check to consider explicit default textdomain usage a warning

[felixarntz]

Fixes #713.

Note: There's no way to do this via WPCS flags, so we need to rely on the error message to detect the relevant violation and downgrade it to a warning accordingly. This can be done with minimal additional code by overriding the relevant parent method.

[github-actions]

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: felixarntz <[email protected]>
Co-authored-by: swissspidy <[email protected]>
Co-authored-by: ernilambar <[email protected]>
Co-authored-by: frantorres <[email protected]>
Co-authored-by: davidperezgar <[email protected]>
Co-authored-by: ironprogrammer <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

[swissspidy]

Still not a fan of projects doing this but this is better than the current experience I suppose.

[ernilambar]

Still not a fan of projects doing this but this is better than the current experience I suppose.

Agree. Personally I would want my plugin's string independent from core string and have full control to the plugin itself.