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.

Sign up for GitHub

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

Jump to bottom

[10.0.x] NO-ISSUE: remove incubator-kie-kogito-serverless-operator from pipelines #1226

Merged
merged 1 commit into from
Jul 17, 2024
Merged

[10.0.x] NO-ISSUE: remove incubator-kie-kogito-serverless-operator from pipelines #1226

merged 1 commit into from
Jul 17, 2024

Conversation

jstastny-cz
Copy link
Contributor

Removing incubator-kie-kogito-serverless-operator pipeline references from kogito-pipelines config.

rodrigonull
rodrigonull approved these changes Jul 17, 2024
View reviewed changes
Copy link
Contributor

@rodrigonull rodrigonull left a comment

Choose a reason for hiding this comment

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

LGTM.

wmedvede
wmedvede reviewed Jul 17, 2024
View reviewed changes
Copy link

@wmedvede wmedvede left a comment

Choose a reason for hiding this comment

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

silly question, do this exclude

wmedvede
wmedvede requested changes Jul 17, 2024
View reviewed changes
Copy link

@wmedvede wmedvede left a comment

Choose a reason for hiding this comment

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

Question, do this remove also exclude this repo from having the CI checks?

@jstastny-cz
Copy link
Contributor Author

@wmedvede on 10.0.x - yes, but that branch shouldn't have existed in serverless operator in the first place (if it's not part of the release).

@tiagobento tiagobento merged commit 302b285 into apache:10.0.x Jul 17, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wmedvede wmedvede wmedvede approved these changes

@rodrigonull rodrigonull rodrigonull approved these changes

@ricardozanini ricardozanini Awaiting requested review from ricardozanini

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jstastny-cz @wmedvede @rodrigonull @tiagobento