google / google/benchmark

[FR] Deprecate `CSVReporter`

Open
#1,413 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
10.4k
Forks
1.8k
Avg merge
2d 4h
Merged PRs (30d)
8

Description

**Is your feature request related to a problem? Please describe.**
A description of how to deprecate `CSVReporter` once and for all.

**Describe the solution you'd like**
* change "output_test_helper.cc" to parse JSON rather than CSV
* add a [tool to parse the JSON output into a CSV](https://gist.github.com/dominichamon/fd0570afb67907be49228f5fff3b33f8)
* remove `CSVReporter` and all references to it (just make it build and pass tests)

**Describe alternatives you've considered**
not deprecating it?

**Additional context**
this would cause a major version bump to v2. we have a v2 branch in which some of the above has been attempted but it is very stale.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.