-
Notifications
You must be signed in to change notification settings - Fork 161
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
option for turning on/off integrations #1776
Comments
Couldn't it be something that is controlled in the comments of the file itself? Similar to other keywords like: I would prefer having it in the source than having to type it on the command line every time I execute the script. |
@maxandersen perhaps this should be part of the build steps feature I had been working on some time ago? There were several ways to select which steps to run, including from the sleeve file. We could then great the quarkus integration as a kind of legacy build step. |
How would that look? |
@maxandersen @quintesse we would be very interested by this feature, since we currently can't execute a client library that is built with quarkus but that has no dependency on quarkus. What can we do to push this forward? |
submit a PR with a flag, maybe
....if wanna keep it simple then just start with |
realized its pretty easy to do. draft at #1875 to try. |
This works perfectly. See: https://github.com/orgs/jbangdev/discussions/1774#discussioncomment-11761212 Thank you very much. |
as discussed in #1774 there can be situatons where the classpath contains integrations you dont want to run.
we should at a minimum have a global off switch, maybe
--no-integrations
where--integrations
are default ?possibly consider
--integrations=none
and--integrations=all
is default and open up for allowing list of '.' names to turn on/off specific integrations?The text was updated successfully, but these errors were encountered: