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

@knative-automation knative-automation commented Jun 2, 2021

Cron -knative-prow-robot

/cc knative-sandbox/channel-wg-leads
/assign knative-sandbox/channel-wg-leads

Produced by: knative-sandbox/knobots/actions/update-deps

Details:

bumping knative.dev/eventing e27695b...608a7fe:
  > 608a7fe upgrade to latest dependencies (# 5447)
bumping knative.dev/networking 6ce35e5...9c0fc00:
  > 9c0fc00 upgrade to latest dependencies (# 430)
  > a442bb6 upgrade to latest dependencies (# 429)
bumping go.uber.org/zap 404189c...ebebbf3:
  > ebebbf3 Release v1.17.0 (# 956)
  > 75ac0de Prepare release v1.17.0
  > f612082 zapcore/FieldType: Don't change enum values (# 955)
  > d3092aa Integrate FOSSA (# 953)
  > 4110cb4 lint: Check that 'go mod tidy' was run (# 951)
  > 756fb2a go mod tidy (# 947)
  > 2732714 internal/readme: Simplify if condition (# 945)
  > 2f8e331 FAQ: Add zapfilter (# 939)
  > 34ea13c Update dependencies to fix vulnerabilities in (# 936)
  > 9a9e13f Switch from Travis to GitHub Actions (# 940)
  > 15dca18 zapcore: Cleanup copy in NewMultiWriteSyncer (# 934)
  > 92549f9 Update dependencies to fix vulnerabilities (# 931)
  > 8d5e39e Add FilterFieldKey to zaptest/observer (# 928)
  > 016a93b Support multi-field encoding using zap.Inline (# 912)
  > 6f4d093 Add support for grpclog.LoggerV2 (# 881)
  > b73d9b5 tools: Move to submodule (# 914)
  > d5dd706 Remove outdated information, make dependencies is no longer necessary. (# 915)
  > dca7c25 Optimize Sugar logger for calls with a single string arg (# 913)
  > 9f3c0ee Fix: code quality issues using DeepSource (# 907)
  > 864bed1 http: support additional content type (# 903)
  > a68efdb travis: Test against ppc64le (# 872)
  > f5a3592 Byte alignment, optimization from 96 to 80 byte (# 865)
  > 5b4722d README: use pkg.go.dev instead of godoc.org (# 883)
  > 4b2b07c field/error: Handle panic in Error() (# 867)
  > 495658f travis: Test against stable and oldstable (# 882)
  > 9d2e5de Add FAQ question on zap dropping logs due to sampling (# 875)
bumping knative.dev/pkg a2869ae...0e61d67:
  > 0e61d67 Bump assorted dependencies (# 2140)
bumping knative.dev/reconciler-test f667a8f...db062eb:
  > db062eb Update actions (# 206)
  > 4da3714 upgrade to latest dependencies (# 203)
  > f2caf81 Fail from ReceiverDone/SenderDone (# 209)

@google-cla google-cla bot added the cla: yes Indicates the PR's author has signed the CLA. label Jun 2, 2021
@knative-prow-robot knative-prow-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jun 2, 2021
@codecov
Copy link

codecov bot commented Jun 2, 2021

Codecov Report

Merging #681 (30dd302) into main (a1bac8d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #681   +/-   ##
=======================================
  Coverage   75.21%   75.21%           
=======================================
  Files         132      132           
  Lines        5782     5782           
=======================================
  Hits         4349     4349           
  Misses       1213     1213           
  Partials      220      220           

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 a1bac8d...30dd302. Read the comment docs.

@lionelvillard
Copy link
Contributor

/approve
/lgtm

@knative-prow-robot knative-prow-robot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jun 2, 2021
bumping knative.dev/eventing e27695b...608a7fe:
  > 608a7fe upgrade to latest dependencies (# 5447)
bumping knative.dev/networking 6ce35e5...9c0fc00:
  > 9c0fc00 upgrade to latest dependencies (# 430)
  > a442bb6 upgrade to latest dependencies (# 429)
bumping go.uber.org/zap 404189c...ebebbf3:
  > ebebbf3 Release v1.17.0 (# 956)
  > 75ac0de Prepare release v1.17.0
  > f612082 zapcore/FieldType: Don't change enum values (# 955)
  > d3092aa Integrate FOSSA (# 953)
  > 4110cb4 lint: Check that 'go mod tidy' was run (# 951)
  > 756fb2a go mod tidy (# 947)
  > 2732714 internal/readme: Simplify if condition (# 945)
  > 2f8e331 FAQ: Add zapfilter (# 939)
  > 34ea13c Update dependencies to fix vulnerabilities in (# 936)
  > 9a9e13f Switch from Travis to GitHub Actions (# 940)
  > 15dca18 zapcore: Cleanup copy in NewMultiWriteSyncer (# 934)
  > 92549f9 Update dependencies to fix vulnerabilities (# 931)
  > 8d5e39e Add FilterFieldKey to zaptest/observer (# 928)
  > 016a93b Support multi-field encoding using zap.Inline (# 912)
  > 6f4d093 Add support for grpclog.LoggerV2 (# 881)
  > b73d9b5 tools: Move to submodule (# 914)
  > d5dd706 Remove outdated information, make dependencies is no longer necessary. (# 915)
  > dca7c25 Optimize Sugar logger for calls with a single string arg (# 913)
  > 9f3c0ee Fix: code quality issues using DeepSource (# 907)
  > 864bed1 http: support additional content type (# 903)
  > a68efdb travis: Test against ppc64le (# 872)
  > f5a3592 Byte alignment, optimization from 96 to 80 byte (# 865)
  > 5b4722d README: use pkg.go.dev instead of godoc.org (# 883)
  > 4b2b07c field/error: Handle panic in Error() (# 867)
  > 495658f travis: Test against stable and oldstable (# 882)
  > 9d2e5de Add FAQ question on zap dropping logs due to sampling (# 875)
bumping knative.dev/pkg a2869ae...0e61d67:
  > 0e61d67 Bump assorted dependencies (# 2140)
bumping knative.dev/reconciler-test f667a8f...db062eb:
  > db062eb Update actions (# 206)
  > 4da3714 upgrade to latest dependencies (# 203)
  > f2caf81 Fail from ReceiverDone/SenderDone (# 209)

Signed-off-by: Knative Automation <automation@knative.team>
@knative-prow-robot knative-prow-robot removed the lgtm Indicates that a PR is ready to be merged. label Jun 3, 2021
Copy link
Member

@aliok aliok left a comment

Choose a reason for hiding this comment

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

/approve
/lgtm

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

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aliok, knative-automation, lionelvillard

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:
  • OWNERS [aliok,lionelvillard]

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
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