Skip to content
This repository has been archived by the owner on Jul 16, 2021. It is now read-only.

Expose single shot CheckForNewDomains function #1020

Merged
merged 2 commits into from Jul 16, 2018

Conversation

gdbelvin
Copy link
Contributor

Some server frameworks support periodic callbacks directly.

@gdbelvin gdbelvin requested review from phad and daviddrysdale and removed request for phad July 16, 2018 15:41
@codecov
Copy link

codecov bot commented Jul 16, 2018

Codecov Report

Merging #1020 into master will decrease coverage by 0.09%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #1020     +/-   ##
=========================================
- Coverage   64.89%   64.79%   -0.1%     
=========================================
  Files          39       39             
  Lines        2692     2696      +4     
=========================================
  Hits         1747     1747             
- Misses        637      641      +4     
  Partials      308      308
Impacted Files Coverage Δ
core/sequencer/sequencer.go 50% <0%> (-2.5%) ⬇️

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 dd05450...1c04620. Read the comment docs.

@gdbelvin gdbelvin requested a review from jtoohill July 16, 2018 16:34
@gdbelvin gdbelvin merged commit 8929b3e into google:master Jul 16, 2018
@gdbelvin gdbelvin deleted the f/periodic branch July 16, 2018 20:37
gdbelvin added a commit to gdbelvin/keytransparency that referenced this pull request Jul 18, 2018
* master:
  Cleanup domains after integration test (google#1019)
  Expose single shot `CheckForNewDomains` function (google#1020)
  Restrict integration.Env to only grpc clients (google#1018)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants