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

update dependencies (tink) #48

Merged
merged 1 commit into from
Nov 11, 2020
Merged

Conversation

kqdeng
Copy link
Contributor

@kqdeng kqdeng commented Oct 31, 2020

Signed-off-by: Kelly Deng kelly@packet.com

Description

Following the merge of tinkerbell/tink#345, the github/tinkerbell/tink dependency in go.mod needs to be updated to point to the latest master.

Why is this needed

The PR tinkerbell/tink#345 updated the protobuf files and as a result, hegel (still importing an older version of tink) can no longer properly communicate with the latest version of tink.

Fixes: #

How Has This Been Tested?

  • Tests pass
  • Tested manually

How are existing users impacted? What migration steps/scripts do we need?

Checklist:

I have:

  • updated the documentation and/or roadmap (if required)
  • added unit or e2e tests
  • provided instructions on how to upgrade

@kqdeng kqdeng requested a review from a team October 31, 2020 00:39
@kqdeng kqdeng force-pushed the update-dependencies branch 2 times, most recently from 7d5e9c3 to 6f5500d Compare November 9, 2020 21:48
Signed-off-by: Kelly Deng <kelly@packet.com>
@kqdeng kqdeng force-pushed the update-dependencies branch from 6f5500d to f4280ff Compare November 9, 2020 22:37
@codecov
Copy link

codecov bot commented Nov 9, 2020

Codecov Report

Merging #48 (f4280ff) into master (1583c90) will not change coverage.
The diff coverage is 50.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #48   +/-   ##
=======================================
  Coverage   24.40%   24.40%           
=======================================
  Files           5        5           
  Lines         504      504           
=======================================
  Hits          123      123           
  Misses        359      359           
  Partials       22       22           
Impacted Files Coverage Δ
grpc_server.go 22.53% <50.00%> (ø)

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 1583c90...f4280ff. Read the comment docs.

@gianarb gianarb added the ready-to-merge Signal to Mergify to merge the PR. label Nov 11, 2020
@mergify mergify bot merged commit c17b512 into tinkerbell:master Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Signal to Mergify to merge the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants