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

File watching using notify #1310

Merged
merged 5 commits into from
Sep 2, 2022
Merged

File watching using notify #1310

merged 5 commits into from
Sep 2, 2022

Conversation

extrawurst
Copy link
Owner

see #1

@extrawurst extrawurst added this to the v0.22 milestone Sep 1, 2022
@extrawurst extrawurst merged commit cf9ce9d into master Sep 2, 2022
@extrawurst extrawurst deleted the file-watching-using-notify branch September 2, 2022 07:59
@cruessler cruessler mentioned this pull request Apr 17, 2023
4 tasks
cruessler added a commit to cruessler/gitui that referenced this pull request Apr 17, 2023
This commit reintroduces code that was previously removed in favor of a
notify-based update trigger. It turned out that notify-based updates can
cause issues in larger repositories, so tick-based updates seemed like a
safer default.

extrawurst#1444
extrawurst#1310
extrawurst pushed a commit that referenced this pull request Apr 29, 2023
* Default to tick-based updates

This commit reintroduces code that was previously removed in favor of a
notify-based update trigger. It turned out that notify-based updates can
cause issues in larger repositories, so tick-based updates seemed like a
safer default.

#1444
#1310

* Add FAQ entry for --watcher

* Remove --poll
IndianBoy42 pushed a commit to IndianBoy42/gitui that referenced this pull request Jun 4, 2024
IndianBoy42 pushed a commit to IndianBoy42/gitui that referenced this pull request Jun 4, 2024
* Default to tick-based updates

This commit reintroduces code that was previously removed in favor of a
notify-based update trigger. It turned out that notify-based updates can
cause issues in larger repositories, so tick-based updates seemed like a
safer default.

extrawurst#1444
extrawurst#1310

* Add FAQ entry for --watcher

* Remove --poll
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.

None yet

1 participant