Added coverage_ignore_file option to combining_builder

[darwin-morocho]

This PR allows to use a new option coverage_ignore_file in the combining_builder with this all generated files will have the next comment at the beginning

// coverage:ignore-file

This is neccessary due tojson_serializable needs this to allow us exclude the generated files from code coverage. For more info check google/json_serializable.dart#1106

[kevmoo]

Have you tried "preamble builder" - #625

I think it's better to use the general solution than to add another one-off.

Let me know if that's enough!

[darwin-morocho]

Have you tried "preamble builder" - #625

I think it's better to use the general solution than to add another one-off.

Let me know if that's enough!

Yes, I do. But json_serializable doesn't support that option yet

[kevmoo]

https://github.com/google/json_serializable.dart/blob/58db73b4c2c3a0c393e4e7a68f499d28051908ac/json_serializable/build.yaml#L5-L10

It doesn't need to. You can configure source_gen separately. That way we don't have to plumb everything through.

[darwin-morocho]

https://github.com/google/json_serializable.dart/blob/58db73b4c2c3a0c393e4e7a68f499d28051908ac/json_serializable/build.yaml#L5-L10

It doesn't need to. You can configure source_gen separately. That way we don't have to plumb everything through.

Amazing. I will try it

[kevmoo]

Closing out. You can reopen if things don't work.

[patrickhammond]

Preamble worked perfectly! Relevant build.yaml configuration below:

targets:
  $default:
    builders:
      source_gen:combining_builder:
        options:
          preamble: |
            // coverage:ignore-file