Skip to content
This repository has been archived by the owner on Sep 2, 2024. It is now read-only.

Conversation

knative-automation
Copy link
Contributor

Produced via:

./hack/update-deps.sh --upgrade
./hack/update-codegen.sh

Cron. -knative-prow-robot
/cc knative-sandbox/channel-wg-leads
/assign knative-sandbox/channel-wg-leads

Signed-off-by: Knative Automation <automation@knative.team>
@google-cla google-cla bot added the cla: yes Indicates the PR's author has signed the CLA. label Nov 26, 2020
@knative-prow-robot knative-prow-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Nov 26, 2020
@codecov
Copy link

codecov bot commented Nov 26, 2020

Codecov Report

Merging #234 (03348ea) into master (dcda191) will increase coverage by 0.16%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #234      +/-   ##
==========================================
+ Coverage   75.62%   75.78%   +0.16%     
==========================================
  Files         112      112              
  Lines        4274     4274              
==========================================
+ Hits         3232     3239       +7     
+ Misses        848      841       -7     
  Partials      194      194              
Impacted Files Coverage Δ
...el/distributed/dispatcher/dispatcher/dispatcher.go 75.53% <0.00%> (+7.44%) ⬆️

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 dcda191...03348ea. Read the comment docs.

Copy link
Contributor

@matzew matzew left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

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

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: knative-automation, 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 Nov 26, 2020
@knative-prow-robot knative-prow-robot merged commit 17fd7e7 into knative-extensions:master Nov 26, 2020
skonto added a commit to skonto/eventing-kafka that referenced this pull request Jun 16, 2021
…ons#234)

* patch vendor

* Support eventing metrics (knative-extensions#688)

* support eventing metrics

* lint

* imports

* update with latest deps

* use transformers in dispatchWithRetries instead of copying

* fix use of transformers

* updates

* pass transformers to executeRequest
matzew pushed a commit to matzew/eventing-kafka that referenced this pull request Jun 21, 2021
…ons#234)

* patch vendor

* Support eventing metrics (knative-extensions#688)

* support eventing metrics

* lint

* imports

* update with latest deps

* use transformers in dispatchWithRetries instead of copying

* fix use of transformers

* updates

* pass transformers to executeRequest
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/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants