Skip to content

Conversation

@dinhxuanvu
Copy link
Member

@dinhxuanvu dinhxuanvu commented Dec 10, 2021

Update operator-framework/api dependency to include a new constraint
type (PR-202 in api).

Signed-off-by: Vu Dinh vudinh@outlook.com

Description of the change:

Motivation for the change:

Reviewer Checklist

  • Implementation matches the proposed design, or proposal is updated to match implementation
  • Sufficient unit test coverage
  • Sufficient end-to-end test coverage
  • Docs updated or added to /docs
  • Commit messages sensible and descriptive

Update operator-framework/api dependency to include a new constraint
type (operator-framework#202).

Signed-off-by: Vu Dinh <vudinh@outlook.com>
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 10, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dinhxuanvu

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 10, 2021
@codecov
Copy link

codecov bot commented Dec 10, 2021

Codecov Report

Merging #880 (8a0dba1) into master (aa2afc9) will not change coverage.
The diff coverage is n/a.

❗ Current head 8a0dba1 differs from pull request most recent head b5ced9b. Consider uploading reports for the commit b5ced9b to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master     #880   +/-   ##
=======================================
  Coverage   52.06%   52.06%           
=======================================
  Files         103      103           
  Lines        9069     9069           
=======================================
  Hits         4722     4722           
  Misses       3440     3440           
  Partials      907      907           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update aa2afc9...b5ced9b. Read the comment docs.

@estroz
Copy link
Member

estroz commented Dec 10, 2021

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Dec 10, 2021
@openshift-merge-robot openshift-merge-robot merged commit 808de07 into operator-framework:master Dec 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants