fixup! feedback applied

Author: virkt25

src/mixins/README.md

@@ -10,8 +10,14 @@ LoopBack 4 supports mixins at an `Application` level. Your partial class can the
 
 ### High level example
 ```ts
-const MixedClass = MyMixinClass(Application);
-const MultipleMixedClassesClass = MyMixinClass(MyMixinClass2(Application));
+class MyApplication extends MyMixinClass(Application) {
+  // Your code
+};
+
+// Multiple Classes mixed together
+class MyApp extends MyMixinClass(MyMixinClass2(Application)) {
+  // Your code
+}
 ```
 
 
@@ -25,11 +31,11 @@ For hello-extensions we write a simple Mixin that allows the `Application` class
 #### An example Logger
 ```ts
 class ColorLogger implements Logger {
-  log(...args: any[]) {
+  log(...args: LogArgs) {
     console.log('log  :', ...args);
   }
 
-  error(...args: any[]) {
+  error(...args: LogArgs) {
     const data = args.join(' ');
     // log in red color
     console.log('\x1b[31m error: ' + data + '\x1b[0m');
@@ -47,15 +53,22 @@ It is also important for the constructor to call `super(args)` so `Application`
 ```ts
 constructor(...args: any[]) {
   super(args);
-}```
+}
+```
 
 ### Binding via `ApplicationOptions`
 As mentioned earlier, since our `args` represents `ApplicationOptions`, we can make it possible for users to pass in their `Logger` implementations in a `loggers` array on `ApplicationOptions`. We can then read the array and automatically bind these for the user. 
 
 #### Example user experience
 ```ts
-new LoggerMixin(Application)({
-  loggers: [ColorLogger],
+class MyApp extends LoggerMixin(Application){
+  constructor(...args: any[]) {
+    super(...args);
+  }
+};
+
+const app = new MyApp({
+  loggers: [ColorLogger]
 });
 ```
 
@@ -109,19 +122,15 @@ mountComponentLoggers(component: Constructor<any>) {
 class LoggingApplication extends LoggerMixin(Application) {
   constructor(...args: any[]) {
     super(...args);
-      const app = this;
+    this.logger(ColorLogger);
   }
-
-  app.logger(ColorLogger);
 }
 
 // Binding a Logger provided by a component
 class MyComponent implements Component{
   loggers: [ColorLogger];
 }
 
-const LoggingApplication = LoggerMixin(Application);
-
 const app = new LoggingApplication({
   components: [MyComponent] // Logger from MyComponent will be bound to loggers.ColorLogger
 });

src/types.ts

@@ -8,10 +8,9 @@
 // tslint:disable-next-line:no-any
 export type LogArgs = any[];
 
+// A traditional Logger interface would have `.warn()` and `.info()` methods &
+// potentially others but they have been omitted to keep this example simple.
 export interface Logger {
   log(...args: LogArgs): void;
   error(...args: LogArgs): void;
-
-  info?(...args: LogArgs): void;
-  warn?(...args: LogArgs): void;
 }