-
Notifications
You must be signed in to change notification settings - Fork 182
BXMSDOC-6656 RN update DTAnalysis wired plugin (Kie7 RN) #2901
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
BXMSDOC-6656 RN update DTAnalysis wired plugin (Kie7 RN) #2901
Conversation
@hmanwani-rh I'll take it this time, and will apply the same to Kogito. Will also see if I can update the docs somewhere relevant also. Probably tomorrow. |
thanks @sterobin |
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.
@tarilabs @hmanwani-rh I pushed a commit to expand the docs around this for community and enterprise with a new "Configurable DMN validation" section. I also aligned the structures of the previous and new release notes just for consistency. Will do the same for Kogito when @tarilabs has that one.
Links:
- Enterprise doc preview - @hmanwani-rh I decided to make this a con instead of a proc because it was more consistent in context and was too clunky as a simple one-step proc.
- Community doc preview - Same
- Release note - @hmanwani-rh You can use this for the enterprise RN when the time comes.
Jenkins retest this please |
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.
Good for me, but I believe as you edited the 7.34 Release Notes you inadvertently copied a paragraph twice, below?
...doc/ReleaseNotes/ReleaseNotesDrools.7.34.0.Final/kie-dmn-validation-in-maven-plugin-con.adoc
Show resolved
Hide resolved
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.
@sterobin Looks good to me, thanks
e12ee67
to
11bf452
Compare
11bf452
to
ecca80f
Compare
@hmanwani-rh Please note that this is for master only because it applies to 7.10, NOT 7.9 (for community release 7.45 and beyond). So this release note and doc update will be available to you post 7.9. If you do any other 7.9 to the DMN content, just note that this PR content will not be carried over from master when you cherry-pick subsequent changes. Let me know if anything about this is unclear. Thanks. |
@sterobin Got it, thanks for letting me know. |
Co-authored-by: sterobin <[email protected]>
Co-authored-by: sterobin <[email protected]>
Co-authored-by: sterobin <[email protected]>
Co-authored-by: sterobin <[email protected]>
Co-authored-by: sterobin <[email protected]>
https://issues.redhat.com/browse/BXMSDOC-6656
this first PR addresses the need for the RN update on the Drools/Kie7 side