don't permit multiple models with the same name

[drewbanin]

• at most one enabled model can occupy a given model name in a run

[drewbanin]

We can add a check for duplicate models here. If there are duplicated model names, we should raise a compiler error

[mturzanska]

I could work on this one over the weekend. Anyone working on it already / any blockers ?

[drewbanin]

@mturzanska nope, no one is working on this, but @jthandy and I just encountered it again today :)

The only catch is that multiple models can exist with the same name, but only one of them can be enabled. Each node adheres to this contract.

It would be great to write a descriptive error message out to the user. Something like Error: model 'events' is duplicated works, but something like:

Error:
Found 2 models with the same name:
 - in package snowplow, models/base/events.sql
 - in package stripe: models/transformed/events.sql

(or similar) would be really great! New error functions can be created here.

Let me know if there's anything I can do to help!

[drewbanin]

This is going out in 0.9.1