-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Moved saved_state/compareGenerateSql.. directory from java source to resources bundle #5424
Conversation
@tati-qalified - @rursprung Guys when you have a chance, could you please try pulling this branch and then running either Thanks, |
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.
Hey @MalloD12 - don't you need to change the path at AbstractVerifyTest.java:39 to point to this new path?
Yeap, I think you are right @filipelautert. I didn't update it because all tests were successfully passing, but it makes total sense to update that path. I'll do it and make sure all tests keep passing. Thanks, |
Done! |
55ba394
to
5c19b1e
Compare
@MalloD12 tested - seems to work as expected! |
i guess in this case i don't need to try this as well since it should reproduce the same way for all of us? :) |
112943f
to
350ffbb
Compare
As discussed recently to avoid confusion on people we have decided to move
saved_state/compareGeneratedSqlWithExpectedSqlForMinimalChangesets
SQL files from the java source directory and place them all in the resources bundle.Fixes #5335
Impact
Description
Things to be aware of
Things to worry about
Additional Context