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

Revert "JBPM-7887 - Use enum in ProcessConfig.setRuntimeStrategy() me… #1625

Merged
merged 1 commit into from Nov 3, 2018

Conversation

cristianonicolai
Copy link
Contributor

…thod"

This reverts commit 2be9414.

@cristianonicolai
Copy link
Contributor Author

Jenkins execute full downstream build

@rsynek
Copy link
Collaborator

rsynek commented Nov 3, 2018

@cristianonicolai Looking at the latest build failure, the FDB for this particular repository is probably wrongly configured - it's missing activation of "wildfly11" profile (although this profile now uses WFLY14).

As the change you're reverting does not seem to be connected with any particular requirement, +1 for reverting, if it unblocks the build.

Copy link
Collaborator

@rsynek rsynek left a comment

Choose a reason for hiding this comment

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

Let's wait at least for PR check to finish before merging this.

@krisv
Copy link
Member

krisv commented Nov 3, 2018

Merging to unblock master. These test failures however need to be looked at, they do not seem to be false positives (but most likely unrelated to this change).

@krisv krisv merged commit 9f2b981 into kiegroup:master Nov 3, 2018
@cristianonicolai cristianonicolai deleted the revert_2 branch November 4, 2018 10:23
MarianMacik added a commit to MarianMacik/droolsjbpm-integration that referenced this pull request Nov 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants