Skip to content

feat: fail on missing operators#257

Merged
rnetser merged 18 commits intoopendatahub-io:mainfrom
rnetser:fail_if
Apr 28, 2025
Merged

feat: fail on missing operators#257
rnetser merged 18 commits intoopendatahub-io:mainfrom
rnetser:fail_if

Conversation

@rnetser
Copy link
Copy Markdown
Contributor

@rnetser rnetser commented Apr 17, 2025

Description

How Has This Been Tested?

Merge criteria:

  • The commits are squashed in a cohesive manner and have meaningful messages.
  • Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
  • The developer has manually tested the changes and verified that the changes work

@github-actions
Copy link
Copy Markdown

The following are automatically added/executed:

  • PR size label.
  • Run pre-commit
  • Run tox
  • Add PR author as the PR assignee

Available user actions:

  • To mark a PR as WIP, add /wip in a comment. To remove it from the PR comment /wip cancel to the PR.
  • To block merging of a PR, add /hold in a comment. To un-block merging of PR comment /hold cancel.
  • To mark a PR as approved, add /lgtm in a comment. To remove, add /lgtm cancel.
    lgtm label removed on each new commit push.
  • To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
    verified label removed on each new commit push.
  • To Cherry-pick a merged PR /cherry-pick <target_branch_name> to the PR. If <target_branch_name> is valid,
    and the current PR is merged, a cherry-picked PR would be created and linked to the current PR.
Supported labels

{'/verified', '/wip', '/hold', '/lgtm'}

lugi0
lugi0 previously approved these changes Apr 17, 2025
@rnetser
Copy link
Copy Markdown
Contributor Author

rnetser commented Apr 17, 2025

/verified

@rnetser
Copy link
Copy Markdown
Contributor Author

rnetser commented Apr 17, 2025

/verified

jiridanek
jiridanek previously approved these changes Apr 17, 2025
mwaykole
mwaykole previously approved these changes Apr 17, 2025
lugi0
lugi0 previously approved these changes Apr 17, 2025
mwaykole
mwaykole previously approved these changes Apr 23, 2025
Copy link
Copy Markdown
Member

@mwaykole mwaykole left a comment

Choose a reason for hiding this comment

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

/lgtm

adolfo-ab
adolfo-ab previously approved these changes Apr 24, 2025
@rnetser
Copy link
Copy Markdown
Contributor Author

rnetser commented Apr 25, 2025

/verified

@opendatahub-tests-bot opendatahub-tests-bot added the Verified Verified pr in Jenkins label Apr 25, 2025
@rnetser
Copy link
Copy Markdown
Contributor Author

rnetser commented Apr 28, 2025

/verified

@github-actions
Copy link
Copy Markdown

Status of building tag latest: success.
Status of pushing tag latest to image registry: success.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants