thiagoelg commented on issue #1183: URL: https://github.com/apache/incubator-kie-issues/issues/1183#issuecomment-2103358892
> ok, what I like about option 3 is that doesn't force all the kie-tools users to install jbang if they are not interested in running the tests. The drawback is that users interested in running those tests (and our CI-CD) must manually install it. That is the same approach we're using with Java. Somehow the `dmn-marshaller-backend-compatibility-tester` is an indirect dependency of `online-editor` and probably many other packages, so most kie-tools users will probably need jbang. If we want to rewrite the jbang install script, it should be easy (the [jbang library](https://github.com/jbangdev/jbang-npm) is really simple), but I personally don't like to install global binaries or call install scripts without the developer knowledge or permission, by just running `pnpm boostrap`. Maybe we could install the binary locally? `kogito-serverless-operator` installs required binaries inside a `./bin` directory on the package itself and then executes them via their path (`./bin/binaryName arg1 arg2`), instead of adding them to the global PATH environment variable. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
