Is PoolTeam.pool redundant?

quizbowl / schema

This is how we describe quizbowl.

https://schema.quizbowl.technology

MIT License

9 stars 3 forks source link

Closed jonahgreenthal closed 4 years ago

jonahgreenthal commented 4 years ago

A Pool contains 0 or more PoolTeams. A PoolTeam has a reference back to Pool. The latter is unnecessary, right?

jonahgreenthal commented 4 years ago

(@rhentzel pointed this out)

puls commented 4 years ago

Redundant, sure. Probably just an artifact about visualizing this in our heads as a join table.

jonahgreenthal commented 4 years ago

Can we remove it without having to do a major version bump? That also seems, in a soft sense, unnecessary.

puls commented 4 years ago

Not if you want to strictly adhere to semantic versioning. But would removing it break anything anywhere? I doubt it.

jonahgreenthal commented 4 years ago

It won't break anything. I'm inclined to just kill it and bump the minor version.