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

OptaPlanner: Fail fast if no constraint implementations are found #9372

Merged
merged 1 commit into from
May 18, 2020

Conversation

ge0ffrey
Copy link
Contributor

This is unrelated to the issue that if no optaplanner annotated classes exist, the extension should do nothing. This does NOT make that issue worse (nor fixes it, that fix is coming later).

@geoand
Copy link
Contributor

geoand commented May 16, 2020

So should this be considered WIP or should it be merged and then we have another PR for the proper fix?

@ge0ffrey
Copy link
Contributor Author

ge0ffrey commented May 18, 2020

@geoand This is a standalone PR ready to merge.
(It's completely unrelated to the other work. I didn't mean to cause confusion with that work.)

Copy link
Contributor

@geoand geoand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK

@geoand geoand merged commit cc8421f into quarkusio:master May 18, 2020
@ge0ffrey
Copy link
Contributor Author

Thanks :)

@geoand geoand added this to the 1.5.0 milestone May 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants