Skip to content
This repository was archived by the owner on Oct 11, 2024. It is now read-only.

Comments

Remove objecthash dependency#1441

Merged
gdbelvin merged 1 commit intogoogle:masterfrom
gdbelvin:rm_obj
Jan 23, 2020
Merged

Remove objecthash dependency#1441
gdbelvin merged 1 commit intogoogle:masterfrom
gdbelvin:rm_obj

Conversation

@gdbelvin
Copy link
Contributor

The last use of object hash was used to pin the config in the
gobindclient. Rather than hashing the config, future versions of the
client should either build in a static config, or supply it direclty

The last use of object hash was used to pin the config in the
gobindclient. Rather than hashing the config, future versions of the
client should either build in a static config, or supply it direclty
@gdbelvin gdbelvin requested a review from a team as a code owner January 23, 2020 10:39
@gdbelvin gdbelvin requested a review from Mercurrent January 23, 2020 10:39
@codecov
Copy link

codecov bot commented Jan 23, 2020

Codecov Report

Merging #1441 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1441   +/-   ##
=======================================
  Coverage   66.31%   66.31%           
=======================================
  Files          54       54           
  Lines        4026     4026           
=======================================
  Hits         2670     2670           
  Misses        962      962           
  Partials      394      394

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 ad57ca8...5b9fbca. Read the comment docs.

@gdbelvin gdbelvin merged commit d8831a2 into google:master Jan 23, 2020
@gdbelvin gdbelvin deleted the rm_obj branch January 23, 2020 11:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants