Skip to content
This repository has been archived by the owner on Mar 2, 2022. It is now read-only.

Aggregate fixes #109

Merged
merged 3 commits into from
Mar 25, 2018
Merged

Aggregate fixes #109

merged 3 commits into from
Mar 25, 2018

Conversation

aagbsn
Copy link
Collaborator

@aagbsn aagbsn commented Mar 25, 2018

relays that aren't currently in the consensus raise a TorProtocolError; this changes the behavior to skip these relays while processing.

@coveralls
Copy link

Coverage Status

Coverage remained the same at ?% when pulling f6e2931 on aggregate_fixes into 5ccbdfc on develop.

@juga0
Copy link
Collaborator

juga0 commented Mar 25, 2018

LGTM, tests raising #94, but i'd merge and fix #94

@juga0 juga0 merged commit daf23e6 into develop Mar 25, 2018
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.

3 participants