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 examples/tiny.html? #230

Closed
RyanGlScott opened this issue Nov 14, 2020 · 0 comments · Fixed by #231
Closed

Remove examples/tiny.html? #230

RyanGlScott opened this issue Nov 14, 2020 · 0 comments · Fixed by #231

Comments

@RyanGlScott
Copy link
Member

Back in 2011, a tiny.html file was checked in. However, it has gone completely unchanged since then, and once #229 lands, it will be out of date with respect to the way criterion plots data. I'd consider regenerating tiny.html, but I can't even find the source code that created it! My best guess is that tiny.html was created from a one-off criterion program that has since been lost to the sands of time.

I suppose we could consider keeping tiny.html around for the sake of being an example criterion HTML report, but there are already examples in the www/ directory that serve the same purpose. Better yet, the examples in www/ correspond to actual, version-controlled examples, so they are actually maintainable. In light of this, I don't see much of a reason to keep tiny.html around. I propose just ripping it out.

RyanGlScott added a commit that referenced this issue Nov 14, 2020
As explained in #230, `tiny.html` isn't serving a useful purpose anymore,
and since I can't find the original `criterion` program that generated it,
it is unmaintainble. Let's just remove it.

Fixes #230.
RyanGlScott added a commit that referenced this issue Nov 14, 2020
As explained in #230, `tiny.html` isn't serving a useful purpose anymore,
and since I can't find the original `criterion` program that generated it,
it is unmaintainble. Let's just remove it.

Fixes #230.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant