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

Upgrade deps #580

Merged
Merged

Conversation

lionelvillard
Copy link
Contributor

@lionelvillard lionelvillard commented Apr 27, 2021

knative/eventing#5317 (comment) follow up.

Proposed Changes

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

Release Note


Docs

@lionelvillard lionelvillard requested review from a team as code owners April 27, 2021 18:41
@lionelvillard lionelvillard requested a review from a user April 27, 2021 18:41
@google-cla google-cla bot added the cla: yes Indicates the PR's author has signed the CLA. label Apr 27, 2021
@knative-prow-robot knative-prow-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Apr 27, 2021
@lionelvillard lionelvillard requested a review from a team April 27, 2021 19:00
@codecov
Copy link

codecov bot commented Apr 27, 2021

Codecov Report

Merging #580 (890fc4e) into main (eb52a20) will increase coverage by 0.12%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #580      +/-   ##
==========================================
+ Coverage   71.56%   71.69%   +0.12%     
==========================================
  Files         137      137              
  Lines        5645     5645              
==========================================
+ Hits         4040     4047       +7     
+ Misses       1378     1371       -7     
  Partials      227      227              
Impacted Files Coverage Δ
pkg/source/reconciler/source/controller.go 0.00% <ø> (ø)
...el/distributed/dispatcher/dispatcher/dispatcher.go 80.53% <0.00%> (+6.19%) ⬆️

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 eb52a20...890fc4e. Read the comment docs.

@devguyio
Copy link
Contributor

/lgtm

@knative-prow-robot knative-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Apr 28, 2021
@knative-prow-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: devguyio
To complete the pull request process, please ask for approval from lionelvillard after the PR has been reviewed.

The full list of commands accepted by this bot can be found 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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
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.

3 participants