-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feat/allow checkers to pass #304
Conversation
Still need to update tests |
const factCheckerCount = totalVoteRequestsCount - errorCount //don't count "error" votes in number of fact checkers, as this will slow the replies unnecessarily. | ||
const validResponsesCount = responsesCount - errorCount //can remove in future and replace with nonErrorCount | ||
const factCheckerCount = totalVoteRequestsCount - passCount //don't count "error" votes in number of fact checkers, as this will slow the replies unnecessarily. | ||
const validResponsesCount = responsesCount - passCount //can remove in future and replace with nonErrorCount |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similar to L63, should we include a check? Defaulting to 0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think no need actually. cos L63 is the only resolved promise in the promise.all that isn't coming from the async getCount function. The getCount function will definitely return an integer >= 0. So i think its fine.
Issues