Skip to content

fix: Debounce filesystem events #1409

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
May 31, 2021
Merged

fix: Debounce filesystem events #1409

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
237f348
debounce svn info calls on filesystem events
May 14, 2021
b888bd7
fixed style issues
Blashaq May 16, 2021
086b2a2
reverted style changes caused by wrong dependencies
Blashaq May 16, 2021
0bdac3c
fixed style issues
Blashaq May 16, 2021
2355965
Merge branch 'master' into master
Blashaq May 17, 2021
98d84f7
Merge branch 'master' into master
Blashaq May 18, 2021
df818f3
Merge branch 'master' into master
Blashaq May 19, 2021
86497da
Merge branch 'master' into master
Blashaq May 28, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 7 additions & 4 deletions src/repository.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -188,12 +188,9 @@ export class Repository implements IRemoteRepository {
this.disposables.push(this._fsWatcher);

this._fsWatcher.onDidAny(this.onFSChange, this, this.disposables);

// TODO on svn switch event fired two times since two files were changed
this._fsWatcher.onDidSvnAny(
async (e: Uri) => {
await this.repository.updateInfo();
this._onDidChangeRepository.fire(e);
await this.onDidAnyFileChanged(e);
},
this,
this.disposables
Expand Down Expand Up @@ -292,6 +289,12 @@ export class Repository implements IRemoteRepository {
);
}

@debounce(1000)
private async onDidAnyFileChanged(e: Uri) {
await this.repository.updateInfo();
this._onDidChangeRepository.fire(e);
}

private createRemoteChangedInterval() {
const updateFreq = configuration.get<number>(
"remoteChanges.checkFrequency",
Expand Down