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

fake_node_reader: Remove unused field #1631

Merged
merged 1 commit into from
May 23, 2019

Conversation

pav-kv
Copy link
Contributor

@pav-kv pav-kv commented May 23, 2019

No description provided.

@codecov
Copy link

codecov bot commented May 23, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1631   +/-   ##
=======================================
  Coverage   67.81%   67.81%           
=======================================
  Files         111      111           
  Lines        9539     9539           
=======================================
  Hits         6469     6469           
  Misses       2378     2378           
  Partials      692      692

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 1b7918a...835ab67. Read the comment docs.

@pav-kv pav-kv merged commit 580ab86 into google:master May 23, 2019
@pav-kv pav-kv deleted the remove_unused_field branch May 23, 2019 15:04
gdbelvin added a commit to gdbelvin/trillian that referenced this pull request May 23, 2019
* master:
  Complete TODO (google#1632)
  fake_node_reader: Remove unused field (google#1631)
  Parallelize VerifyMapLeavesResponse (google#1615)
  Remove redundant root hash calculations (google#1630)
  sequencer: Consolidate compact.Tree initialization (google#1629)
gdbelvin added a commit that referenced this pull request May 23, 2019
* master: (94 commits)
  Complete TODO (#1632)
  fake_node_reader: Remove unused field (#1631)
  Parallelize VerifyMapLeavesResponse (#1615)
  Remove redundant root hash calculations (#1630)
  sequencer: Consolidate compact.Tree initialization (#1629)
  Remove unused NodeReader method (#1625)
  Fix bazel build (#1627)
  added/updated postgresql implementation of log_storage (#1571)
  Clean up compact.Tree tests (#1622)
  Remove old hash list format from compact.Tree (#1621)
  Mention protoc-gen-doc in README.md
  merkle: Add testonly package with golden hashes (#1620)
  compact: Simplify getting hashes in NewTreeWithState (#1618)
  compact: Implement Tree using Range (#1522)
  Guard verbose logging in merkle path code (#1604)
  Make MaskLeft of NodeID return an explicit copy. (#1612)
  Convert directly from Int.Bits() to NodeID (#1614)
  Couple of changes to make NodeIDs more frugal. (#1613)
  compact.Tree: Change semantic of adding leaves (#1592)
  MapHasher: Do not return error from HashLeaf (#1611)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants