Vetalice 最近的时间轴更新
Vetalice

Vetalice

V2EX 第 636965 号会员,加入于 2023-07-03 19:37:25 +08:00
Vetalice 最近回复了
发现那个小伙已经很积极地在我的 PR 下帮忙 review 了,可能是我太多心了
@majula 感谢分享,理想中确实是这样做最好。我这个问题有点麻烦的是工作上我们确实很快就要用,等不及他慢慢完成,而且我们提交 PR 前后脚不超过半天,我知道对方在做这个还是我问这个 feature 的上游依赖作者的时候他告诉我的,而且当时我们已经完成得差不多了。发现他的 PR 后也给他发过邮件,当时的想法是他可以保留我们共通的部分,然后把这边已经额外做好的合进去。只是那个资深贡献者也是只花了半天就 review 了并且在他那边刚上班就留下了建议。

你提醒了我至少可以在 Acknowledgment 感谢一下,即使不管他做的工作,他也提示过我上游在未来有个比较重要且影响到这个功能的优化要注意。
@msg7086 道理是这样,所以如果我是他我也会干脆放弃掉自己的 PR 。只是之前邮件往来对方非常热情地介绍了自己的工作,不想打击到他热情,毕竟从他 Github 的介绍看来他还挺珍惜这个第一份实习。

@leonshaw 其实因为目前的实现还有些边角问题需要处理所以他也可以提一些 commit 。但是我不知道他是怎么想的,他的实习公司又会怎么安排所以不好自作多情。在别人建议他放弃之前我给他发过邮件提议合并我们的实现,我认为我们共同已经实现的一半差不多,另一半则是我这边更好。先等他回复吧,虽然不确定他还回不回……
308 天前
回复了 klmd99 创建的主题 Rust [求助] VSCode 写 rust 不报错了,怎么解决
Ctrl+Shift+M 看看。可能是因为 Cargo 解析参数配置不对导致 check 依赖的时候直接在依赖里报错中断。如果确认参数没问题可以 cargo clean 再 restart analyzer 试试。
关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   968 人在线   最高记录 6679   ·     Select Language
创意工作者们的社区
World is powered by solitude
VERSION: 3.9.8.5 · 12ms · UTC 20:32 · PVG 04:32 · LAX 13:32 · JFK 16:32
Developed with CodeLauncher
♥ Do have faith in what you're doing.