Skip to content
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.

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

fix: Added debounce delay for saving indexes #183

Merged
merged 1 commit into from
Oct 10, 2022

Conversation

agg23
Copy link
Contributor

@agg23 agg23 commented Sep 28, 2022

The process of saving indexes to the workspace can be very slow, so performing this immediately on every change can freeze VSCode. This change adds a 10 second debounce period, where once no activity has occurred for 10 seconds, the save will begin.

@agg23 agg23 force-pushed the agg23/IndexSaveDebounce branch from 714d3bf to dc0c7d1 Compare September 30, 2022 03:27
@joecrop
Copy link
Collaborator

joecrop commented Oct 5, 2022

@agg23 please rebase master here too and let's hope the pipeline passes on this.

@agg23 agg23 force-pushed the agg23/IndexSaveDebounce branch from dc0c7d1 to 7c39f32 Compare October 5, 2022 14:00
@agg23 agg23 requested a review from joecrop October 5, 2022 14:02
@joecrop
Copy link
Collaborator

joecrop commented Oct 9, 2022

Hooray! The pipeline is running again! Looks like just a couple lint violations to resolve and we can get it merged.

@agg23 agg23 force-pushed the agg23/IndexSaveDebounce branch from 7c39f32 to 8dc0b30 Compare October 9, 2022 18:01
@agg23 agg23 force-pushed the agg23/IndexSaveDebounce branch from 8dc0b30 to a417b5f Compare October 10, 2022 13:48
@agg23
Copy link
Contributor Author

agg23 commented Oct 10, 2022

Sorry about that. I amended and force pushed, but didn't actually stage the change, so nothing actually changed.

@joecrop joecrop merged commit 58963e7 into eirikpre:master Oct 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants