V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
XiLemon
V2EX  ›  程序员

Phabricator CR 工作流程

  •  
  •   XiLemon · 53 天前 · 657 次点击
    这是一个创建于 53 天前的主题,其中的信息可能已经有所发展或是发生改变。

    大概看了一下这篇文章: https://secure.phabricator.com/phame/post/view/766/write_review_merge_publish_phabricator_review_workflow/

    Phabricator 是先 review 最后再合并成一个 commit 提交到远端,但是我想的是保留多个 commit,这样可以有一个清晰的历史,请问这个该怎么操作呢?

    有熟悉用 Phabricator 做 CR 的 v2er 么,说一说最佳实践呗 :-)

    7 条回复    2021-01-07 11:32:06 +08:00
    micookie
        1
    micookie   53 天前
    比如你 arc diff 的时候提交了 5 次

    arc land 的时候会自动读取你 5 次的 git log,写到这个提交的 log 中

    我大概记得是这样
    CEBBCAT
        2
    CEBBCAT   53 天前 via Android
    暂时想不到,等下查查文档。我觉得可以先开通一个特殊权限或者规则规避 reject
    avastms
        3
    avastms   53 天前
    太难了,规格太高! 不 review 不能进代码库,这起码是两个人干一个人的活。。
    XiLemon
        4
    XiLemon   53 天前
    @micookie #1 arc diff 5 次是会生成 5 个 Differential 么,我还在摸索这个工具怎么用呢
    @CEBBCAT #2 老哥啥意思呢
    @avastms #3 我觉得 review 过得代码才能进代码库是合理的,可能是我的用法不对,把一个 feat 的代码整成了一个 commit,我是想保留一些历史记录的。成本确实有些高,但我觉得是值得的。可惜感觉公司内部并没有正确的使用这个工具,至少我问过几个老同事,他们也没能说清楚 -_-||
    wph95
        5
    wph95   53 天前
    > 但是我想的是保留多个 commit,这样可以有一个清晰的历史
    无法理解。
    1. 想保存 commit 来展示各个功能?
    按 Phabricator 的逻辑和一些实践。一个 小功能 一个 diff,diff 大了就应当拆成多个。
    2. 想保存 commit 展示开发过程和相关上下文?
    这不管是 Phabricator,还是 Github,项目大了必然是开启 squash 的,一个 PR/Diff 的开发过程汇入主分支要 squash,没人在主分支关心你的开发上下文。要是想关心,去看 PR/Diff 单个具体历史记录呗。留着污染主分支干啥。
    micookie
        6
    micookie   53 天前
    @XiLemon

    arc diff 5 次会产生 5 次 commit 到同一个 revision(评审单)下,推送到 phabricator 的仓库,不会推送到远端仓库。

    最后 arc land 的时候才会把 5 次的提交记录合并成 1 次 commit 推送到远端仓库。

    phabricator 认为,一个 revision 是一个需求,既然是同一个需求的提交记录,应该合并为同一次 commit 。

    即使你想查看每次的 commit 的话,可以在 phabricator 页面上查看,在远端仓库是不会保存详细的提交的。
    XiLemon
        7
    XiLemon   52 天前
    @wph95 #5
    @micookie #6

    看来是该转变一下思路了,再摸索摸索,谢谢二位 ^_^!
    关于   ·   帮助文档   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   5305 人在线   最高记录 5497   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 23ms · UTC 03:06 · PVG 11:06 · LAX 19:06 · JFK 22:06
    ♥ Do have faith in what you're doing.