Skip to content
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

[7.7.x] AF-1079,PLANNER-1091 Fix SolverConfig validation #274

Merged
merged 1 commit into from May 4, 2018

Conversation

Projects
None yet
2 participants
@Rikkola
Copy link
Member

commented May 4, 2018

(cherry picked from commit 43001f8)

https://issues.jboss.org/browse/PLANNER-1091

@yurloc Here is the fix for 7.7.x in case you did not have one prepared.

@Rikkola

This comment has been minimized.

Copy link
Member Author

commented May 4, 2018

I'll do 6.5.x too, but might have to do it on Monday and ask for acks.

@yurloc yurloc self-requested a review May 4, 2018

@yurloc yurloc changed the title AF-1079,PLANNER-1091 Fix SolverConfig validation (#273) AF-1079,PLANNER-1091 Fix SolverConfig validation May 4, 2018

@yurloc

yurloc approved these changes May 4, 2018

@Rikkola

This comment has been minimized.

Copy link
Member Author

commented May 4, 2018

+1 from me. The fix is orinally made by @yurloc. The build is green so I think we should merge this.

@yurloc yurloc merged commit 5ce72c1 into kiegroup:7.7.x May 4, 2018

1 check passed

Linux Build finished. 285 tests run, 0 skipped, 0 failed.
Details

@Rikkola Rikkola deleted the Rikkola:PLANNER-1091-validation branch May 4, 2018

@yurloc yurloc changed the title AF-1079,PLANNER-1091 Fix SolverConfig validation [7.7.x] AF-1079,PLANNER-1091 Fix SolverConfig validation May 4, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.