breautek commented on issue #526: URL: https://github.com/apache/cordova-cli/issues/526#issuecomment-683805909
> I think it's debatable if the Java version is actually needed. After all we just re-assured ourselves that nothing except Java 8 will work anyway sweat_smile. Yeah, but I mean, if it helps us pinpoint an environment flaw, I think it's still good to include. > Regarding the node modules dependency tree: running npm ls takes very long usually, so I would prefer to avoid running it as part of cordova info. If needed, users can easily provide that information separately. Perhaps. I just learned about the `npm ls` command myself. Just have to I guess tell people to either try clearing their `package-lock.json` / `node_modules` tree and reinstalling to ensure all their dependencies are up to date, especially when they have weird issues that aren't reproducible. > Since cordova-android is the only part of Cordova that uses Java, Java version information should be provided by cordova-android's upcoming API method getEnvironmentInfo. Agreed. And I think our consensus is the only thing that should be added is Java, so perhaps this issue should be closed and a new issue should be created in `cordova-android`? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
