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

Fix table of supported libraries after the 9.0.1 release #78

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

martinweiler
Copy link
Contributor

We should add a table for each release in order to be precise on which library versions we support (as mandated by the bom).

@tiagobento is Kie Tools version 0.30.0 correct for both 9.0.0 and 9.0.1?

@LightGuard
Copy link
Member

@porcelli whenever you get a chance to take a look at this.

@LightGuard
Copy link
Member

@porcelli or @tiagobento how does this look?

@LightGuard
Copy link
Member

Bump for @porcelli and @tiagobento. Please review so we can merge.

|Kogito runtimes| 1.40.2.Final
|Kogito add-ons| 1.40.2.Final
|Drools | 8.40.1.Final
|KIE Tools | 0.30.0
Copy link
Contributor

Choose a reason for hiding this comment

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

KIE Tools is not directly supported.

Comment on lines +33 to +35
|Kogito runtimes| 1.40.2.Final
|Kogito add-ons| 1.40.2.Final
|Drools | 8.40.1.Final
Copy link
Contributor

Choose a reason for hiding this comment

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

it's not all libraries from those repos that we support. I think we have to articulate ore clear that we support a subset of libraries - and for those the versions are the ones listed here.

@LightGuard
Copy link
Member

@martinweiler Alex had some changes requested, I don't want to lose this.

@porcelli
Copy link
Contributor

this looks stale, should be merged or delete? @martinweiler @LightGuard

@LightGuard
Copy link
Member

Is for 9.0.1, but it might be stale for that

@martinweiler
Copy link
Contributor Author

I think it still makes sense to push this. The points @porcelli has raised are valid, but go beyond what this PR tried to achieve. The goal was to make it clear that 9.0.0 and 9.0.1 consist of different library versions.

@LightGuard
Copy link
Member

Still valid to merge?

@martinweiler
Copy link
Contributor Author

With the release of 9.1.x and the changes to the supported environments page in general, this is of lesser importance now.

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.

3 participants