src-d / style-analyzer

Lookout Style Analyzer: fixing code formatting and typos during code reviews
GNU Affero General Public License v3.0
32 stars 21 forks source link

Create report system for typos analyzer #704

Closed zurk closed 5 years ago

zurk commented 5 years ago

Related PR https://github.com/src-d/style-analyzer/pull/703 First in a batch.

zurk commented 5 years ago

Waiting for https://github.com/src-d/style-analyzer/pull/709 to be merged as it is an important one. While waiting create General report system.

zurk commented 5 years ago

Status: PR https://github.com/src-d/style-analyzer/pull/721 is ready to be reviewed but first:

  1. 708 and https://github.com/src-d/style-analyzer/pull/725 should be merged

  2. Updated typos corector model should be loaded by @irinakhismatullina (current master does not work)

To get report:

  1. Merge https://github.com/src-d/style-analyzer/pull/721
  2. Waiting @vmarkovtsev to fix commit hash in the dataset to one where typo was created.
  3. Ask @EgorBu to filter dataset one time more after that and push it to repo.
  4. Come up with metrics and report info we want to get.
  5. Code proder report generation.
  6. Generate report.
  7. Push report.

While I am waiting I can:

  1. Add tests
  2. Add CLI
  3. Add makefile command.