-
Notifications
You must be signed in to change notification settings - Fork 125
chore: move quickstarts to declarative shadow variables #902
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
chore: move quickstarts to declarative shadow variables #902
Conversation
a59972d
to
4429a35
Compare
Requires TimefoldAI/timefold-solver#1739 |
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.
We're getting a little bit ahead of ourselves here.
To put DSV in quickstarts, they first need to be promoted to public API.
To do that, there are still some outstanding issues.
No point in merging this before those happen, as this will all change anyway. (No preview, different imports, maybe even different names.)
a5ee7f7
to
07aac73
Compare
...test/java/org/acme/projectjobschedule/solver/ProjectJobSchedulingConstraintProviderTest.java
Show resolved
Hide resolved
One food-packaging ShadowVariable could not be migrated, since it uses a non-declarative shadow on non-declarative shadow as a source.
07aac73
to
1cbafac
Compare
Enterprise needs updates it seems. |
Description of the change
Change usages of VariableListeners and CascadingShadowVariable to declarative shadow variables.
Checklist
Development
Code Review