Add a CombiningBuilder for .g.dart files and document how to use it

[natebosch]

We added runs_before and applies_builder to build config so it should be possible now to put everything together with one or more source_gen part builders cooperating to generate a single part file. We'll want a "CombiningBuilder" ("MergingBuilder"?) in this package to read in all the files generated by different part builders and write them to a single output.

• Fix duplicate headers - Fixes for CombiningBuilder #348
• Fix logging of missing part directive - Fixes for CombiningBuilder #348
• Document usage in readme
• Update build.yaml to properly expose CombiningBuilder
• BLOCKING Get build issue fixed related to running built-to-cache builder over transitive dependencies - Filter out the builders which "applies_builders" some builder which is "build_to: source" on non root packages build#1578
• Add a cleanup builder

After published...

• Add a working example to repo/example

[kevmoo]

@grouma – added some work items before we can close this. The first two are handled by my pending PR

[kevmoo]

How are we doing w/ the check boxes here @natebosch and @grouma ?

[kevmoo]

...still need to tweak build.yaml, right?

[natebosch]

...still need to tweak build.yaml, right?

I think it looks correct (other than a nit about single vs double quotes). README and example are the remaining issues. I wanted also to validate usage which is what google/json_serializable.dart#260 is meant to accomplish.

[kevmoo]

ack

…

On Thu, Jul 19, 2018 at 9:45 AM Nate Bosch ***@***.***> wrote: ...still need to tweak build.yaml, right? I think it looks correct (other than a nit about single vs double quotes). README and example are the remaining issues. I wanted also to validate usage which is what google/json_serializable.dart#260 <google/json_serializable.dart#260> is meant to accomplish. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#319 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AABCir3Xw-YG0rAS8prOWR_tmhzHz74_ks5uILfAgaJpZM4Sy-EV> .

[kevmoo]

@natebosch – I think we're don here, right?

For the example, I want to add it after this package is published so it can refer to published packages and be easier to copy-paste

[natebosch]

yes, I think we are safe to publish now. @grouma, @jakemac53 - can you think of anything else blocking?

[jakemac53]

I haven't been following this to closely 😬 but nothing blocking I am aware of.

[kevmoo]

Version bumb to v1 first?

…

On Mon, Jul 23, 2018, 12:49 Jacob MacDonald ***@***.***> wrote: I haven't been following this to closely 😬 but nothing blocking I am aware of. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#319 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AABCiqfZ5UAyyn_gS3JWcQkbVi7ujQh1ks5uJii1gaJpZM4Sy-EV> .

[grouma]

Should be good to go.

[natebosch]

There's an example now, I think we've finished.