Merge pull request #20429 from emberjs/consistent-return

[BUGFIX stable] TS lint-level error in `mixins/-proxy.ts`

packages/@ember/-internals/runtime/lib/mixins/-proxy.ts

@@ -119,9 +119,7 @@ const ProxyMixin = Mixin.create({
 
   unknownProperty(key: string) {
     let content = contentFor(this);
-    if (content) {
-      return get(content, key);
-    }
+    return content ? get(content, key) : undefined;
   },
 
   setUnknownProperty(key: string, value: unknown) {

packages/@ember/debug/lib/deprecate.ts

@@ -116,6 +116,7 @@ if (DEBUG) {
     captureErrorForStack = () => {
       try {
         __fail__.fail();
+        return;
       } catch (e) {
         return e;
       }

packages/@ember/object/observable.ts

@@ -329,7 +329,7 @@ interface Observable {
     Remove an observer you have previously registered on this object. Pass
     the same key, target, and method you passed to `addObserver()` and your
     target will no longer receive notifications.
-    
+
     @method removeObserver
     @param {String} key The key to observe
     @param {Object} target The target object to invoke
@@ -400,7 +400,7 @@ interface Observable {
     This allows you to inspect the value of a computed property
     without accidentally invoking it if it is intended to be
     generated lazily.
-    
+
     @method cacheFor
     @param {String} keyName
     @return {Object} The cached value of the computed property, if any
@@ -526,10 +526,7 @@ const Observable = Mixin.create({
 
   cacheFor(keyName: string) {
     let meta = peekMeta(this);
-
-    if (meta !== null) {
-      return meta.valueFor(keyName);
-    }
+    return meta !== null ? meta.valueFor(keyName) : undefined;
   },
 });