Skip to content

Conversation

@yaovi-a
Copy link
Contributor

@yaovi-a yaovi-a commented Sep 22, 2021

Creating my first PR of many others: there'll be other unsolicited grammar changes.

@CLAassistant
Copy link

CLAassistant commented Sep 22, 2021

CLA assistant check
All committers have signed the CLA.

@tsachiherman
Copy link
Contributor

please sign the CLAassistant

@codecov-commenter
Copy link

codecov-commenter commented Sep 22, 2021

Codecov Report

Merging #2938 (19a9744) into master (e4842fc) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2938      +/-   ##
==========================================
- Coverage   47.30%   47.28%   -0.02%     
==========================================
  Files         356      356              
  Lines       57244    57244              
==========================================
- Hits        27081    27070      -11     
- Misses      27098    27106       +8     
- Partials     3065     3068       +3     
Impacted Files Coverage Δ
ledger/blockqueue.go 82.18% <0.00%> (-1.73%) ⬇️
catchup/peerSelector.go 98.95% <0.00%> (-1.05%) ⬇️
catchup/service.go 68.57% <0.00%> (-0.78%) ⬇️
ledger/acctupdates.go 62.50% <0.00%> (-0.60%) ⬇️
network/wsPeer.go 74.93% <0.00%> (+0.55%) ⬆️
cmd/tealdbg/debugger.go 73.86% <0.00%> (+1.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 e4842fc...19a9744. Read the comment docs.

@yaovi-a
Copy link
Contributor Author

yaovi-a commented Sep 22, 2021

Just signed it.

@tsachiherman tsachiherman merged commit e69b10d into algorand:master Sep 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants