[GitHub] brodybits commented on issue #390: WIP: major eslint updates (with eslint-config-standard@next) for evaluation only

[GitBox <gi...@apache.org>]

brodybits commented on issue #390: WIP: major eslint updates (with eslint-config-standard@next) for evaluation only
URL: https://github.com/apache/cordova-ios/pull/390#issuecomment-410268966
 
 
   To clarify the general question: I would generally favor upgrading to "latest" version of dependencies and devDependencies whenever possible when targeting upcoming major release. The problem is that using eslint@latest would lead to the ugly warning as discussed in the updated description and review comments unless we would use eslint-config-standard@next (eslint-config-standard@^12.0.0-alpha.0). We are basically stuck waiting for eslint-config-standard@12.0.0 to be released as discussed in standard/eslint-config-standard#123.
   
   I think we should not introduce "@next" dependency version unless absolutely necessary for some reason.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org