-
Notifications
You must be signed in to change notification settings - Fork 29
Backport of #185 #186
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
Backport of #185 #186
Conversation
…ce-import-uuid-behavior Added option to control the uuid-behavior
@dbu this should be the last one :) thanks for merging, tagging and just everything ;) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, just one request to change how we clean up the command argument in the test.
please add a new commit for that in your branch, so that the git history between 1.2 and 1.3 stays consistent.
@@ -94,6 +94,7 @@ public function testNodeUpdate($options) | |||
$this->setupQueryManager($options); | |||
|
|||
$args = [ | |||
'--query-language' => 'jcr-sql2', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the original solution was to completely drop this argument: b222c2a . for the 1.2 branch i think it was simply never fixed, i think we can just leave it as is there. but can you please delete it here?
the thing is: sql2 is default, but if you specify a query language option, you have to specify a value for it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dbu should be fixed
tagged 1.3.3 with this |
@dbu thanks |
No description provided.