You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
(This is not a current concern for me; we ended up setting up a build that doesn't use gradle-conjure, but noticed this during the development process. I felt it was worth documenting here.)
It's common for Gradle builds involving Node to bootstrap it so the user doesn't need to have a correct version of it installed on their machine (e.g. as supported by gradle-node-plugin).
However, using gradle-conjure to generate, build, and publish a TypeScript package will run node and npm from the default PATH, with no way to specify another location AFAICT. (This includes the invocation of the TypeScript generator, for which the entry point is a Node.js script.) This requires developers and CI images to have Node already installed, and in theory could lead to different results across machines if different versions are installed.
It would be nice if the location of the node and npm executables to use could be configured, so they could be pointed at a bootstrapped installation.
The text was updated successfully, but these errors were encountered:
I just hit this as well as I'm working on a PR and running tests. It is a pain to have things fail out of the box for unknown reasons. At least a comment in the CONTRIBUTING doc would be good.
(This is not a current concern for me; we ended up setting up a build that doesn't use gradle-conjure, but noticed this during the development process. I felt it was worth documenting here.)
It's common for Gradle builds involving Node to bootstrap it so the user doesn't need to have a correct version of it installed on their machine (e.g. as supported by gradle-node-plugin).
However, using gradle-conjure to generate, build, and publish a TypeScript package will run
node
andnpm
from the default PATH, with no way to specify another location AFAICT. (This includes the invocation of the TypeScript generator, for which the entry point is a Node.js script.) This requires developers and CI images to have Node already installed, and in theory could lead to different results across machines if different versions are installed.It would be nice if the location of the
node
andnpm
executables to use could be configured, so they could be pointed at a bootstrapped installation.The text was updated successfully, but these errors were encountered: