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

在实现discover时候watch方法一直提示不符合签名 #390

Open
brian-god opened this issue Mar 15, 2024 · 10 comments
Open

在实现discover时候watch方法一直提示不符合签名 #390

brian-god opened this issue Mar 15, 2024 · 10 comments

Comments

@brian-god
Copy link

但是我经过多次对比,并没有找到区别
j1710495626320
j1710495746042

@brian-god
Copy link
Author

我也尝试指定自己定义的结构体为key,但还是能解决,求大佬帮忙
1710499217086

@PureWhiteWu
Copy link
Member

你看下你的 async_broadcast 的版本是否一致

@brian-god
Copy link
Author

brian-god commented Mar 18, 2024

你看下你的 async_broadcast 的版本是否一致

大佬您好,我看了一下,版本都是0.7

@brian-god
Copy link
Author

aaaa1710726272722

@PureWhiteWu
Copy link
Member

你好,你可以把 async-broadcast 版本降低到 0.6 再试试看,非 main 分支上的版本依赖的还是 0.6

@Millione
Copy link
Member

@PureWhiteWu 我们是不是把 async-broadcast 的依赖版本改成用 *或0 比较好,这样应该就可以保证和用户依赖到的版本一致?

@PureWhiteWu
Copy link
Member

@Millione 不太合适,主要原因有几个:

  1. 带有版本为 * 的依赖是无法 publish 到 crates 上的;
  2. 写 0 的话,不能保证用户也用的 “0”,而且这样也很容易产生 break change;
  3. 更好的方案还是 re-export 一下

@PureWhiteWu
Copy link
Member

或者可以看看这里能不能改成用 tokio 的实现

@iamazy
Copy link

iamazy commented Mar 18, 2024

@Millione 不太合适,主要原因有几个:

  1. 带有版本为 * 的依赖是无法 publish 到 crates 上的;
  2. 写 0 的话,不能保证用户也用的 “0”,而且这样也很容易产生 break change;
  3. 更好的方案还是 re-export 一下

改为 async-broadcast=">=0.6"

@PureWhiteWu
Copy link
Member

@iamazy 还是问题 2,比如如果 async-broadcast 发了一个版本 0.8 产生了 break change,那用户就会编译失败。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

4 participants