Skip to content

Commit

Permalink
fix(closure-compiler): adds nocollapse to static members (#3519)
Browse files Browse the repository at this point in the history
  • Loading branch information
benlesh authored Apr 5, 2018
1 parent b3a657d commit 8758a5d
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 1 deletion.
1 change: 1 addition & 0 deletions src/MiscJSDoc.ts
Original file line number Diff line number Diff line change
Expand Up @@ -133,6 +133,7 @@ export class ObservableDoc {
* @static true
* @name create
* @owner Observable
* @nocollapse
*/
static create<T>(onSubscription: <R>(observer: Observer<R>) => TeardownLogic): Observable<T> {
return new Observable<T>(onSubscription);
Expand Down
4 changes: 3 additions & 1 deletion src/internal/observable/SubscribeOnObservable.ts
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,12 @@ export interface DispatchArg<T> {
* @hide true
*/
export class SubscribeOnObservable<T> extends Observable<T> {
/** @nocollapse */
static create<T>(source: Observable<T>, delay: number = 0, scheduler: SchedulerLike = asap): Observable<T> {
return new SubscribeOnObservable(source, delay, scheduler);
}

/** @nocollapse */
static dispatch<T>(this: SchedulerAction<T>, arg: DispatchArg<T>): Subscription {
const { source, subscriber } = arg;
return this.add(source.subscribe(subscriber));
Expand All @@ -46,4 +48,4 @@ export class SubscribeOnObservable<T> extends Observable<T> {
source, subscriber
});
}
}
}
1 change: 1 addition & 0 deletions src/internal/observable/dom/AjaxObservable.ts
Original file line number Diff line number Diff line change
Expand Up @@ -131,6 +131,7 @@ export class AjaxObservable<T> extends Observable<T> {
* @static true
* @name ajax
* @owner Observable
* @nocollapse
*/
static create: AjaxCreationMethod = (() => {
const create: any = (urlOrRequest: string | AjaxRequest) => {
Expand Down
1 change: 1 addition & 0 deletions src/internal/operators/observeOn.ts
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,7 @@ export class ObserveOnOperator<T> implements Operator<T, T> {
* @extends {Ignored}
*/
export class ObserveOnSubscriber<T> extends Subscriber<T> {
/** @nocollapse */
static dispatch(this: SchedulerAction<ObserveOnMessage>, arg: ObserveOnMessage) {
const { notification, destination } = arg;
notification.observe(destination);
Expand Down
2 changes: 2 additions & 0 deletions src/internal/testing/TestScheduler.ts
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,7 @@ export class TestScheduler extends VirtualTimeScheduler {
}
}

/** @nocollapse */
static parseMarblesAsSubscriptions(marbles: string): SubscriptionLog {
if (typeof marbles !== 'string') {
return new SubscriptionLog(Number.POSITIVE_INFINITY);
Expand Down Expand Up @@ -185,6 +186,7 @@ export class TestScheduler extends VirtualTimeScheduler {
}
}

/** @nocollapse */
static parseMarbles(marbles: string,
values?: any,
errorValue?: any,
Expand Down

0 comments on commit 8758a5d

Please sign in to comment.