Skip to content

Commit

Permalink
Removed call to Observable.cache
Browse files Browse the repository at this point in the history
This method was removed from RxJS-5.0.0 release (ReactiveX/rxjs#2012).

Maybe some caching alternative could be implemented later.
For now, it needs to be removed to make the library usable again
(OmniSharp/omnisharp-atom#880).
  • Loading branch information
igorrafael committed Oct 29, 2016
1 parent 8a4e032 commit 125148e
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions lib/helpers/runtime.ts
Original file line number Diff line number Diff line change
Expand Up @@ -257,9 +257,8 @@ export const isSupportedRuntime = memoize(function (ctx: RuntimeContext) {
.map(x => ({ runtime: Runtime.ClrOrMono, path: [x.path].concat(PATH).join(delimiter) }))
.take(1)
.defaultIfEmpty({ runtime: Runtime.CoreClr, path: process.env.PATH });
})
});
//.do(ct => console.log(`Supported runtime for "${Runtime[ct.runtime]}" was: ${Runtime[ct.runtime]}`))
.cache(1);
}, function ({platform, arch, runtime, version}: RuntimeContext) { return `${arch}-${platform}:${Runtime[runtime]}:${version}`; });

function findOmnisharpExecuable(runtimeId: string, location: string): Observable<boolean> {
Expand Down

0 comments on commit 125148e

Please sign in to comment.