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

什么软件支持在源代码文件的某一行上编写评论

  •  
  •   wuzhouhui · 2021-03-14 11:25:45 +08:00 · 2482 次点击
    这是一个创建于 1383 天前的主题,其中的信息可能已经有所发展或是发生改变。

    code review 通常是在代码提交之前, 但由于种种原因, 不得不是代码合并到库里后, 再去阅读代码. 假设在阅读的过程中, 发现一些问题, 需要针对特定的代码行提点意见, 提完意见后, 代码的作者能够收到邮件通知并回复, 并提交补丁来改正问题. 请问有什么样的工具可能提供这样的功能?

    再明确一下, 不对针对 diff 编写评论, 而是文件的当前内容.

    10 条回复    2021-03-14 19:08:30 +08:00
    locoz
        1
    locoz  
       2021-03-14 11:28:37 +08:00 via Android   ❤️ 2
    这种情况貌似一般是提一个 issue 并带上代码行号,我记得像 GitHub 是可以直接显示对应部分代码。
    oott123
        2
    oott123  
       2021-03-14 11:41:17 +08:00   ❤️ 3
    如果这行代码之后修改了,这个评论如何显示呢?

    就一般的版本控制软件而言,为了避免代码变动之后评论有问题,都只能对某个 commit 上的代码进行评论。
    cluulzz
        3
    cluulzz  
       2021-03-14 13:24:59 +08:00 via iPhone
    //fixme ?
    hhyyd
        4
    hhyyd  
       2021-03-14 13:43:13 +08:00
    codestream
    leaves98
        5
    leaves98  
       2021-03-14 13:49:23 +08:00
    不修改源代码文件进行评论的话,vscode 倒是有个插件 CodeTour
    qinxi
        6
    qinxi  
       2021-03-14 14:03:46 +08:00
    jb 家的 upsource
    forgottencoast
        7
    forgottencoast  
       2021-03-14 17:40:59 +08:00
    这些不是 git 类的源代码管理软件所提供的功能吗?类似 GitHub 之类的都是这样的呀。
    前提是你要采用合适的工作流程。
    比如根据每个 Issue 单独拉出来一个分支,然后修改好以后再提交,合并之前 Code Review,都是针对某一行进行评论的。
    你如果有洁癖,可以在代码合并到主分支之前,把这个 Code Review 所有的 commit 合并,显得干净。
    各种需求通常都可以通过不同的工作流程来解决。
    jmc891205
        8
    jmc891205  
       2021-03-14 18:32:01 +08:00 via iPhone
    我们用的是 Code collaborator
    niubee1
        9
    niubee1  
       2021-03-14 18:59:20 +08:00
    起点也可以
    Kobayashi
        10
    Kobayashi  
       2021-03-14 19:08:30 +08:00 via Android
    开 issue 不就完了吗?描述里添加 URL 链接指向某段代码。
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   1096 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 21ms · UTC 23:01 · PVG 07:01 · LAX 15:01 · JFK 18:01
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.