-
Notifications
You must be signed in to change notification settings - Fork 78
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
Feature request - append for mlBundle #675
Comments
I haven't tried this yet, but I think for modules, this would work:
At runtime, ml-gradle will then modify this to toss |
Good point - just force it to append again. This would make sense for modules. However, I'd be curious about the overhead of ml-config. Will play around with this soon. Trying to see how far I can take using a bundle for various combined-code and config requirements |
It should work for ml-config as well - ml-gradle will merge the resource files together, with the last directory in the list "winning" and overwriting any keys already present in the merged JSON object. |
Hi ROb. Yes this is all working fine from what I can see.
…On Fri, 29 Sept 2023 at 18:02, Rob Rudin ***@***.***> wrote:
It should work for ml-config as well - ml-gradle will merge the resource
files together, with the last directory in the list "winning" and
overwriting any keys already present in the merged JSON object.
—
Reply to this email directly, view it on GitHub
<#675 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABD2VTEBV7YZC7YOMKGXX6TX43WKTANCNFSM6AAAAAA5K4XSCI>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Closing as I believe this request was addressed, but please reopen if that is not the case. |
Referring to this document:
https://github.com/marklogic/ml-gradle/wiki/Bundles
The items are always pre-pended to the configurations - making them get executed first. However, there may be times in which one would want to use a bundle to change a default behavior. Example: replace an xQuery module to change how something was implemented.
It would be nice if, via configuration, the choice could be made between prepend and append
The text was updated successfully, but these errors were encountered: