Skip to content

Commit e643015

Browse files
committed
ref(replay): Prefix private methods/properties with _ in ReplayContainer
1 parent d803287 commit e643015

File tree

5 files changed

+121
-118
lines changed

5 files changed

+121
-118
lines changed

packages/replay/src/index.ts

+6-6
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ export class Replay implements Integration {
3333
/** In tests, this is only called the first time */
3434
protected _hasCalledSetupOnce: boolean = false;
3535

36-
private replay?: ReplayContainer;
36+
private _replay?: ReplayContainer;
3737

3838
constructor({
3939
flushMinDelay = 5000,
@@ -150,30 +150,30 @@ Sentry.init({ replaysOnErrorSampleRate: ${errorSampleRate} })`,
150150
* PerformanceObserver, Recording, Sentry SDK, etc)
151151
*/
152152
start(): void {
153-
if (!this.replay) {
153+
if (!this._replay) {
154154
return;
155155
}
156156

157-
this.replay.start();
157+
this._replay.start();
158158
}
159159

160160
/**
161161
* Currently, this needs to be manually called (e.g. for tests). Sentry SDK
162162
* does not support a teardown
163163
*/
164164
stop(): void {
165-
if (!this.replay) {
165+
if (!this._replay) {
166166
return;
167167
}
168168

169-
this.replay.stop();
169+
this._replay.stop();
170170
}
171171

172172
private _setup(): void {
173173
// Client is not available in constructor, so we need to wait until setupOnce
174174
this._loadReplayOptionsFromClient();
175175

176-
this.replay = new ReplayContainer({
176+
this._replay = new ReplayContainer({
177177
options: this.options,
178178
recordingOptions: this.recordingOptions,
179179
});

0 commit comments

Comments
 (0)