From db260f24625be5526240dea9cd7151d7108f0d97 Mon Sep 17 00:00:00 2001
From: Nicholas Jamieson <nicholas@cartant.com>
Date: Tue, 7 Nov 2017 07:07:12 +1000
Subject: [PATCH] fix(typings): fix subscribe overloads

Remove the no-arg overload. If not removed, any Observable will be
compatible with any ObservableInput regardless of type - as T does not
appear in the no-arg overload.

Closes #3052
---
 src/Observable.ts | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/Observable.ts b/src/Observable.ts
index d09810c8439..74bc9af34f4 100644
--- a/src/Observable.ts
+++ b/src/Observable.ts
@@ -73,8 +73,7 @@ export class Observable<T> implements Subscribable<T> {
     return observable;
   }
 
-  subscribe(): Subscription;
-  subscribe(observer: PartialObserver<T>): Subscription;
+  subscribe(observer?: PartialObserver<T>): Subscription;
   subscribe(next?: (value: T) => void, error?: (error: any) => void, complete?: () => void): Subscription;
   /**
    * Invokes an execution of an Observable and registers Observer handlers for notifications it will emit.