This repository was archived by the owner on Mar 28, 2024. It is now read-only.
上游优先的故事
#12
Replies: 2 comments 3 replies
很认同tison的这段话~概读了通篇文章,收货很大 |
1 reply
|
贡献一个上游优先的还未成功的小故事: 起因是在 engula 里看到 format Cargo.toml 的需求 https://github.com/engula/engula/issues/181 ,用 taplo 作为 workaround 部分实现了之后,我发现上游 rustfmt 已经有一个一年多的这个问题的讨论 rust-lang/rustfmt#4091 。里面讨论了许多包括 taplo 在内的 workaround,以至于 maintainer 出来声明说这个功能是在 rustfmt 的 scope 内的,我们要自己支持,不要再提 workaround 了。只不过迟迟没有人愿意出来实现。 几个月后我突然有闲心,就花了一个周末实现了 rust-lang/rustfmt#5240 。经过一番 review 以后,maintainer 表示实现的挺好,但是因为种种小原因没合进去。之后我精力有限没有继续推进完善,maintainer 也是带宽非常有限,最后打上了 low priority 和 pending review 的 tag,就大半年又过去了。 🤣 |
2 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
https://town.korandoru.io/blog/upstream-first-stories
All reactions