erisu commented on code in PR #1551:
URL: https://github.com/apache/cordova-ios/pull/1551#discussion_r2231209439
##########
lib/Api.js:
##########
@@ -461,7 +461,7 @@ class Api {
if (val) {
podsjsonFile.decrementDeclaration(declaration);
} else {
- const message = util.format('plugin \"%s\"
declaration \"%s\" does not seem to be in pods.json, nothing to remove. Will
attempt to remove from Podfile.', plugin.id, podJson.declaration); /* eslint
no-useless-escape : 0 */
+ const message = util.format('plugin \"%s\"
declaration \"%s\" does not seem to be in pods.json, nothing to remove. Will
attempt to remove from Podfile.', plugin.id, podJson.declaration);
Review Comment:
From a quick search, it looks like `util.format` is used 37 times across 8
files.
Since there's more than just these three lines, I'll submit a separate PR to
refactor them to use template literals.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org
