Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[main] Upgrade to latest dependencies #887

Merged

Conversation

knative-automation
Copy link
Contributor

Cron -knative-prow-robot

/cc knative/serving-writers
/assign knative/serving-writers

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

bumping sigs.k8s.io/yaml 9535b3b...c3772b5:
  > c3772b5 Merge pull request # 97 from liggitt/owners
  > eead467 Merge pull request # 76 from natasha41575/go-yaml-fork
  > 604031b Update owners
  > 3566774 Merge pull request # 96 from liggitt/msg
  > f2c0229 add test demonstrating consistency between yaml.v2 and yaml.v3
  > 65d71bb Merge pull request # 87 from justinsb/spew_no_more
  > 3f79f32 Revert error message changes
  > 20d25fa update LICENSE, owners, and readmes
  > d865f09 Merge pull request # 83 from kubernetes-sigs/bump-version-of-golang-to-1.19-and-drop-older-versions
  > 13f0a74 Replace go-spew with go-cmp
  > a7b5665 exclude forked libs from golint
  > 5303805 Merge pull request # 75 from brianpursley/benchmarks
  > b02c352 reformat to golang 1.19 standards
  > b6d8b45 fix leading newline issue
  > b5bdf49 Merge pull request # 66 from ncdc/remove-ncdc-from-owners
  > 0beda2b Add benchmarks
  > 2f8489c Bump version of golang to 1.19 and prune older versions
  > 330e1e5 make sequence style configurable
  > 78c1299 Merge pull request # 69 from ncdc/fix-lint-error
  > 94f1bbd Remove myself (ncdc) from OWNERS
  > 2d151d8 Add benchmarks
  > 70ebea7 change import paths to forked yaml libs
  > 39f74b9 Merge pull request # 65 from amurant/non_functional
  > 5e30182 Fix lint error
  > b96582b fork go-yaml v2 and v3
  > 4f103b1 greatly improve testing framework & added a lot of new tests
  > 3c98e65 expand variable names in yaml.go, add better comments, improve errors, add copyright
  > 4924d91 expand variable names in fields.go
  > 59dc8b4 add 1.16 and 1.17 to github workflows
  > 740b971 remove vendor folder
bumping knative.dev/pkg b7a62c0...b09d4f2:
  > b09d4f2 Bump sigs.k8s.io/yaml from 1.3.0 to 1.4.0 (# 2879)
  > 9f71af7 Bump github.com/google/uuid from 1.3.1 to 1.4.0 (# 2878)
bumping github.com/google/uuid b3cae7c...8de8764:
  > 8de8764 chore(master): release 1.4.0 (# 134)
  > 7c22e97 Clarify the documentation of Parse to state its job is to parse, not validate, strings. (# 135)
  > cd5fbbd feat: UUIDs slice type with Strings() convenience method (# 133)
  > 47f5b39 docs: fix a typo in CONTRIBUTING.md (# 130)
  > 542ddab chore(tests): add Fuzz tests (# 128)
  > 06716f6 chore(tests): Add json.Unmarshal test with empty value cases (# 116)

Signed-off-by: Knative Automation <automation@knative.team>
@knative-automation knative-automation force-pushed the auto-updates/update-deps-main branch from 77e2cd8 to 1af3536 Compare November 2, 2023 01:30
@knative-prow knative-prow bot requested a review from a team November 2, 2023 01:30
@knative-prow knative-prow bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Nov 2, 2023
Copy link

codecov bot commented Nov 2, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (aea9eae) 82.14% compared to head (1af3536) 82.14%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #887   +/-   ##
=======================================
  Coverage   82.14%   82.14%           
=======================================
  Files          45       45           
  Lines        1725     1725           
=======================================
  Hits         1417     1417           
  Misses        266      266           
  Partials       42       42           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@ReToCode ReToCode 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 knative-prow bot added the lgtm Indicates that a PR is ready to be merged. label Nov 2, 2023
Copy link

knative-prow bot commented Nov 2, 2023

[APPROVALNOTIFIER] This PR is APPROVED

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

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 knative-prow bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 2, 2023
@knative-prow knative-prow bot merged commit 3bd0f7d into knative:main Nov 2, 2023
23 checks passed
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. 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.

2 participants