Skip to content

Commit 52e2f08

Browse files
cexbrayatmmalerba
authored andcommitted
refactor(core): remove unused APP_EFFECT_SCHEDULER in effect code (#59679)
The `APP_EFFECT_SCHEDULER` Injection token is never used and not a public API. PR Close #59679
1 parent 2bddce2 commit 52e2f08

2 files changed

Lines changed: 0 additions & 12 deletions

File tree

packages/core/src/render3/reactivity/effect.ts

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ import {
1919
} from '../../../primitives/signals';
2020
import {FLAGS, LViewFlags, LView, EFFECTS} from '../interfaces/view';
2121
import {markAncestorsForTraversal} from '../util/view_utils';
22-
import {InjectionToken} from '../../di/injection_token';
2322
import {inject} from '../../di/injector_compatibility';
2423
import {Injector} from '../../di/injector';
2524
import {assertNotInReactiveContext} from './asserts';
@@ -201,15 +200,6 @@ export interface RootEffectNode extends EffectNode {
201200
scheduler: EffectScheduler;
202201
}
203202

204-
/**
205-
* Not public API, which guarantees `EffectScheduler` only ever comes from the application root
206-
* injector.
207-
*/
208-
export const APP_EFFECT_SCHEDULER = /* @__PURE__ */ new InjectionToken('', {
209-
providedIn: 'root',
210-
factory: () => inject(EffectScheduler),
211-
});
212-
213203
export const BASE_EFFECT_NODE: Omit<EffectNode, 'fn' | 'destroy' | 'injector' | 'notifier'> =
214204
/* @__PURE__ */ (() => ({
215205
...REACTIVE_NODE,

packages/core/src/render3/reactivity/root_effect_scheduler.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,6 @@
77
*/
88

99
import {ɵɵdefineInjectable} from '../../di/interface/defs';
10-
import {PendingTasksInternal} from '../../pending_tasks';
11-
import {inject} from '../../di/injector_compatibility';
1210

1311
/**
1412
* Abstraction that encompasses any kind of effect that can be scheduled.

0 commit comments

Comments
 (0)