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

Update documentation and examples #496

Merged
merged 1 commit into from Aug 10, 2022
Merged

Update documentation and examples #496

merged 1 commit into from Aug 10, 2022

Conversation

arp242
Copy link
Member

@arp242 arp242 commented Aug 8, 2022

Update some documentation and add various examples. I added the examples
as subcommands of ./cmd/fsnotify, so they're directly runnable, and it's
at least guaranteed they compile.

Fixes #49
Fixes #74
Fixes #94
Fixes #122
Fixes #238
Fixes #372
Fixes #401

@arp242 arp242 marked this pull request as draft August 8, 2022 13:59
@arp242 arp242 force-pushed the doc branch 5 times, most recently from 62d3685 to 9c96789 Compare August 9, 2022 15:41
Update some documentation and add various examples. I added the examples
as subcommands of ./cmd/fsnotify, so they're directly runnable, and it's
at least guaranteed they compile.

This adds some simpler test cases so it's easier to verify it actually
works as documented on all platforms, and it adds internal.Debug() for
all platforms, which is useful in dev.

Fixes 49
Fixes 74
Fixes 94
Fixes 122
Fixes 238
Fixes 372
Fixes 401
@arp242 arp242 marked this pull request as ready for review August 9, 2022 17:13
@arp242 arp242 merged commit 61a05ce into main Aug 10, 2022
@arp242 arp242 deleted the doc branch August 10, 2022 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment