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

[NFC] RunBenchmarks(): s/has_repetitions/might_have_aggregates/ #707

Merged
merged 1 commit into from
Oct 18, 2018

Conversation

LebedevRI
Copy link
Collaborator

That is the real purpose of that bool. A follow-up change will
make it consider something else other than repetitions.

That is the real purpose of that bool. A follow-up change will
make it consider something else other than repetitions.
@LebedevRI LebedevRI requested review from dmah42 and EricWF and removed request for dmah42 October 17, 2018 18:18
@coveralls
Copy link

Coverage Status

Coverage remained the same at 89.224% when pulling a71a0f0 on LebedevRI:nfc-has-aggregates into d731697 on google:master.

@AppVeyorBot
Copy link

Build benchmark 1531 completed (commit dcb7aea9f9 by @LebedevRI)

@LebedevRI
Copy link
Collaborator Author

Thank you for the review!

@LebedevRI LebedevRI merged commit 9cacec8 into google:master Oct 18, 2018
@LebedevRI LebedevRI deleted the nfc-has-aggregates branch October 18, 2018 12:03
EricWF pushed a commit to efcs/benchmark that referenced this pull request Nov 29, 2018
…le#707)

That is the real purpose of that bool. A follow-up change will
make it consider something else other than repetitions.
JBakamovic pushed a commit to JBakamovic/benchmark that referenced this pull request Dec 6, 2018
…le#707)

That is the real purpose of that bool. A follow-up change will
make it consider something else other than repetitions.
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.

None yet

5 participants