-
Notifications
You must be signed in to change notification settings - Fork 252
Conversation
You may want to wait until https://codereview.appspot.com/58500043/ is merged before merging this. |
@kardianos @eaburns @mirtchovski @parkghost @romanoff @kylelemons GoDoc indicates that you have a package that imports Please be aware that go.exp is experimental and the API will be changing over the next few weeks. If you don't want to update your package in lock-step, consider importing |
Thanks for the heads up. On Fri Jan 31 2014 at 11:13:39 PM, Nathan Youngman [email protected]
|
I'll drop a note here once https://codereview.appspot.com/58500043/ is merged. Until then I don't recommend importing |
@howeyc Should we try to apply bug fixes to both repositories while only changing the API at go.exp/fsnotify? |
I'm not sure to whom this question is directed, but since I'm CCed, I'll If it's not hard to fix the bugs in both places then it's nice to have On Thu Feb 06 2014 at 9:44:21 AM, Nathan Youngman [email protected]
|
CL 58500043 has been merged. https://codereview.appspot.com/58500043/ @eaburns I agree. Maybe we can fix a few issues before jumping on the API changes. |
Go 1.3 will include os/fsnotify in the standard library with a new API.
code.google.com/p/go.exp/fsnotify
(GoDoc) for the latest API under development.github.com/howeyc/fsnotify
(GoDoc) for the stable API.code.google.com/p/go.exp/fsnotify