Skip to content
This repository has been archived by the owner on Jun 4, 2021. It is now read-only.

Updating deps and setting krshapedlogic=false #1340

Merged
merged 4 commits into from
Jun 25, 2020
Merged

Updating deps and setting krshapedlogic=false #1340

merged 4 commits into from
Jun 25, 2020

Conversation

matzew
Copy link
Member

@matzew matzew commented Jun 25, 2020

Signed-off-by: Matthias Wessendorf mwessend@redhat.com

Fixes #

Proposed Changes

Next

Each source/channel maintainer can decided to actually enable it - but current behavoir matches what we had anyways.

Release Note


Docs

Signed-off-by: Matthias Wessendorf <mwessend@redhat.com>
@knative-prow-robot knative-prow-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jun 25, 2020
@googlebot googlebot added the cla: yes Indicates the PR's author has signed the CLA. label Jun 25, 2020
@knative-prow-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: matzew

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

@knative-prow-robot knative-prow-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 25, 2020
@matzew
Copy link
Member Author

matzew commented Jun 25, 2020

/test pull-knative-eventing-contrib-unit-tests

@devguyio
Copy link
Contributor

/assgin

@devguyio
Copy link
Contributor

/lgtm

@knative-prow-robot knative-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Jun 25, 2020
@devguyio
Copy link
Contributor

/retest

Signed-off-by: Matthias Wessendorf <mwessend@redhat.com>
Signed-off-by: Matthias Wessendorf <mwessend@redhat.com>
@knative-prow-robot knative-prow-robot removed the lgtm Indicates that a PR is ready to be merged. label Jun 25, 2020
Signed-off-by: Matthias Wessendorf <mwessend@redhat.com>
@matzew matzew changed the title Updating deps and stuff Updating deps and setting krshapedlogic=false Jun 25, 2020
@matzew
Copy link
Member Author

matzew commented Jun 25, 2020

This is similar to @whaught did on different repos as well

@devguyio
Copy link
Contributor

/lgtm

@knative-prow-robot knative-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Jun 25, 2020
@knative-prow-robot knative-prow-robot merged commit c31935f into knative:master Jun 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cla: yes Indicates the PR's author has signed the CLA. lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants