-
Notifications
You must be signed in to change notification settings - Fork 40
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
Update default camunda version to 7.2.0 #38
Comments
Shouldn't this be a 2.0 release since we have a lot of api changes? At least thats what I am planning to do on *-mockito and *-needle ... |
Well, I would for now just change the default build dependency for camunda-bpm-assert. Assert does not transitively draw in any camunda bpm version... so you provide your own anyway. And at the moment assert still works with 7.0.0-Final upwards. The moment we really add additional assertions for cmmn et al. I'd see a switch to 2.0... agreed? |
The motivation to change the minimal required version from 7.0 to 7.1 was to get rid of teh extra repositories and use maven central only ... I still do not get why you should build against 7.2 in version 1.2 while you will start to actually use 7.2 features with the 2.0 release ... I mean, it will probably not hurt anyone, but why bother? |
Well... you are right. At the same time camunda-bpm-assert 1.2 will be included into the coming archetypes for 7.2. So I have the slight feeling that it's better to show that "compatibility" with 7.2 in the pom just in case somebody looks. And in the comment it's written there, that assert is fully compatible with 7.0.0-Final upwards. The CI will integrate against all versions all the time, anyway. |
Convinced. |
See Issue #39 of why we should go to 7.2 |
I think I prefer to implement as much assertions as possible with the minimum version necessary to get the particular job done. Therefore, I propose to use Camunda BPM FormService for #39. |
Implicitely resolved by #56. We compile now always with the current release (currently 7.4.0), but test against all releases. |
No description provided.
The text was updated successfully, but these errors were encountered: