[proposal] exclude generated files from code coverage

[iapicca]

Use case

currently generated files negatively affect code coverage

Proposal

add the comment // coverage:ignore-file,
made available by dart tools coverage (see dart-archive/coverage#302),
to the generate files to exclude these from coverage

cc @osa1

[osa1]

Thanks for the ping. Could you give us some context please? What is coverage? How do you run it? What's the problem not having // converage:... causes?

Sorry I replied to the original version of your issue (before the edits). I think this makes sense. I'm on vacation this week but we'll discuss this next week and then I'll update.

[osa1]

Hi @iapicca. We discussed this briefly and coverage devs opened an issue for adding a command line flag to coverage to ignore files: dart-lang/tools#510. This coverage feature is much more general, you can ignore files generated by other code generators as well, and with that feature we don't need to change anything in this library.

[iapicca]

Fair enough, thanks for checking this out @osa1

For anyone that is reading this issue and uses codecov
you may also add to your codecov.yml

ignore:
  - '**/*.pb.dart'