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

Remove All.ns #248

Merged
merged 1 commit into from May 6, 2018

Conversation

2 participants
@smarr
Copy link
Owner

commented May 6, 2018

All.ns is not supported and not used.

The recommended way of running benchmarks is using the codespeed.conf with ReBench.

One thing we might want to think about is to have an the benchmark harnesses provide a list of all available benchmarks.
So, that ReBench could find and execute all benchmarks, even the ones that are not yet part of the configuration.
This would make sure that we execute them all, and there is no bitrot.

Remove All.ns
Signed-off-by: Stefan Marr <git@stefan-marr.de>

@smarr smarr added the enhancement label May 6, 2018

@smarr smarr added this to the v0.7.0 milestone May 6, 2018

@smarr smarr added this to Open Issues in Completeness via automation May 6, 2018

@coveralls

This comment has been minimized.

Copy link

commented May 6, 2018

Coverage Status

Coverage remained the same at 77.403% when pulling 4024d52 on del-ns into 385926d on dev.

@smarr smarr merged commit 7a152c4 into dev May 6, 2018

4 checks passed

Codacy/PR Quality Review Up to standards. A positive pull request.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 77.403%
Details

Completeness automation moved this from Open Issues to Completed May 6, 2018

@smarr smarr deleted the del-ns branch May 6, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.