From 77a4529342ab58c0899e5add35edecb7d1305c8d Mon Sep 17 00:00:00 2001
From: Christopher Vogt <oss.nsp@cvogt.org>
Date: Wed, 15 Feb 2017 21:58:46 -0500
Subject: Document common inheritance pitfalls

---
 README.md | 33 +++++++++++++++++++++++++++++++++
 1 file changed, 33 insertions(+)

diff --git a/README.md b/README.md
index c353503..205cc6e 100644
--- a/README.md
+++ b/README.md
@@ -495,3 +495,36 @@ only show first 20 lines of type errors to catch the root ones
 ```
 cbt c 2>&1 | head -n 20
 ```
+
+Inheritance Pittfalls
+---------------------
+```
+trait Shared extends BaseBuild{
+  // this lowers the type from Seq[Dependency] to Seq[DirectoryDependency]
+  override def dependencies = Seq( DirectoryDependency(...) )
+}
+class Build(...) extends Shared{
+  // this now fails because GitDependency is not a DirectoryDependency
+  override def dependencies = Seq( GitDependency(...) )
+}
+// Solution: raise the type explicitly
+trait Shared extends BaseBuild{
+  // this lowers the type from Seq[Dependency] to Seq[DirectoryDependency]
+  override def dependencies: Seq[Dependency] = Seq( DirectoryDependency(...) )
+}
+```
+
+```
+trait Shared extends BaseBuild{
+  override def dependencies: Seq[Dependency] = Seq() // removes all dependencies, does not inclide super.dependencies
+}
+trait SomePlugin extends BaseBuild{
+  // adds a dependency
+  override def dependencies: Seq[Dependency] = super.dependencies ++ Seq( baz )
+}
+class Build(...) extends Shared with SomePlugin{
+  // dependencies does now contain baz here, which can be surprising
+}
+// Solution can be being careful about the order and using traits instead of classes for mixins
+class Build(...) extends SomePlugin with Shared
+```
-- 
cgit v1.2.3