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

Add FS notify (inotify) functionality #3

Merged
merged 1 commit into from
Jun 19, 2022
Merged

Add FS notify (inotify) functionality #3

merged 1 commit into from
Jun 19, 2022

Conversation

jasonrogena
Copy link
Owner

Add logic for notifying when files in registered directories change.

Signed-off-by: Jason Rogena jason@rogena.me

@codecov-commenter
Copy link

codecov-commenter commented Jun 12, 2022

Codecov Report

Merging #3 (ff63e71) into main (d7bc742) will increase coverage by 1.18%.
The diff coverage is 80.08%.

@@            Coverage Diff             @@
##             main       #3      +/-   ##
==========================================
+ Coverage   78.27%   79.45%   +1.18%     
==========================================
  Files          11       14       +3     
  Lines         405      774     +369     
==========================================
+ Hits          317      615     +298     
- Misses         88      159      +71     
Impacted Files Coverage Δ
src/main.rs 1.17% <0.00%> (-1.21%) ⬇️
src/library/mod.rs 67.47% <67.92%> (+6.61%) ⬆️
src/fs_notify/mod.rs 79.59% <79.59%> (ø)
src/library/mime_type/mod.rs 81.48% <80.00%> (ø)
src/fs_notify/tests_unsupported_os.rs 90.90% <90.90%> (ø)
src/fs_notify/tests_supported_os.rs 97.50% <97.50%> (ø)
src/config/mod.rs 75.00% <100.00%> (+1.31%) ⬆️
src/config/tests.rs 100.00% <100.00%> (ø)
src/library/mime_type/tests.rs 100.00% <100.00%> (ø)
src/library/tests.rs 100.00% <100.00%> (ø)
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d7bc742...ff63e71. Read the comment docs.

@jasonrogena jasonrogena force-pushed the fs_notify branch 12 times, most recently from bc6319a to d71ea84 Compare June 14, 2022 22:08
@jasonrogena jasonrogena force-pushed the fs_notify branch 12 times, most recently from e139d49 to 3f302e9 Compare June 19, 2022 21:35
Add logic for notifying when files in registered directories change
on macOS and Linux. Postpond support on Windows because of #9.

Related to #9

Signed-off-by: Jason Rogena <jason@rogena.me>
@jasonrogena jasonrogena marked this pull request as ready for review June 19, 2022 22:38
@jasonrogena jasonrogena merged commit 19ccd82 into main Jun 19, 2022
@jasonrogena jasonrogena deleted the fs_notify branch June 19, 2022 22:39
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