Skip to content

Comments

Add support for flume#680

Merged
dfaust merged 3 commits intonotify-rs:mainfrom
RenDiego:main
Mar 26, 2025
Merged

Add support for flume#680
dfaust merged 3 commits intonotify-rs:mainfrom
RenDiego:main

Conversation

@RenDiego
Copy link
Contributor

added basic support for flume

@dfaust
Copy link
Member

dfaust commented Mar 26, 2025

Thank you. The code looks good. The only thing that's missing is the documentation.
Basically, everywhere that crossbeam-channel is mentioned, it should also mention flume.
Plus, a change-log entry would be nice.

@RenDiego
Copy link
Contributor Author

I believe that's all,
I assumed the next notify version will be 9.0.0 instead of 8.0.1 and updated the changelog for debouncer-mini and debouncer-full

@dfaust dfaust merged commit e96e0e2 into notify-rs:main Mar 26, 2025
3 checks passed
@dfaust
Copy link
Member

dfaust commented Mar 26, 2025

Thanks again!

JohnTitor pushed a commit that referenced this pull request Jul 3, 2025
* added support for flume

* documentation for flume feature

* update CHANGELOG.md
JohnTitor pushed a commit that referenced this pull request Jan 16, 2026
* added support for flume

* documentation for flume feature

* update CHANGELOG.md
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