• Home
  • Categories
  • WIKI
  • LANGUAGE: en
    • GitHub跟没有似的，干啥都没点回音
    Coordination 844 views · 14 replies ·
    Tofloor
    poster avatar
    rigoligo
    deepin
    2020-12-22 07:32
    Author

    累计发了3个pr，一个都没合并，拖了数月都没合并。没有人看社区的吗？这个issue有人回复我都觉得是谢天谢地了。

    Reply Favorite View the author
    All Replies
    tmzqcn
    deepin
    2020-12-22 08:07
    #1

    支持楼主！

    Reply View the author
    jingle
    deepin
    2020-12-22 16:57
    #2

    issues和pr 都会有人处理，有专门的机器人通知，时间上面可能会有延时。


    Reply View the author
    偕臧
    deepin developers team
    2020-12-22 17:05
    #3

    刚看了一下欢迎，你的这个提的比较早，当时还没有 bot 还每天添加自动添加审查者功能（现在已经有了）， 所以没有人 code review； 这会已经添加对应的审查者了。



    这种长期没有回复的话，可以 @我

    Reply View the author
    偕臧
    deepin developers team
    2020-12-22 17:08
    #4

    且需要楼主重新修改一下提交格式： 提交规范 ， 否则机器人会不通过，自动毙掉（对所有提交提交的人一视同仁（包含公司员工））

    Reply View the author
    rigoligo
    deepin
    2020-12-22 20:28
    #5
    偕臧：

    且需要楼主重新修改一下提交格式： 提交规范 ， 否则机器人会不通过，自动毙掉（对所有提交提交的人一视同仁（包含公司员工））

    不符合规范的话，建议加个提醒

    Reply View the author
    rigoligo
    deepin
    2020-12-22 21:31
    #6

    一次只能一个commit局限太大啦。我修CI要测试多个commit才有结果，然而这多个commit不能用rebase来合并，由于git的commit是一个引用一个，这样会影响父commit。这个强烈建议找个解决方案。

    Reply View the author
    偕臧
    deepin developers team
    2020-12-22 21:32
    #7
    rigoligo：

    不符合规范的话，建议加个提醒

    后面确实会加一个 hook 来做检测，另外相关的提交指南也已经推出（支持 中/英）


    https://docs.deepin.org/pages/3fd76c



    Reply View the author
    偕臧
    deepin developers team
    2020-12-22 21:33
    #8
    rigoligo：

    一次只能一个commit局限太大啦。我修CI要测试多个commit才有结果，然而这多个commit不能用rebase来合并，由于git的commit是一个引用一个，这样会影响父commit。这个强烈建议找个解决方案。

    这个无法解决， gerrit 的自带限制

    Reply View the author
    chenzhiwei
    deepin
    2020-12-22 23:23
    #9

    借楼问下，我这个PR格式规范不？


    https://github.com/linuxdeepin/deepin-terminal/pull/48



    Reply View the author
    sudo_free
    deepin
    2020-12-22 23:49
    #10
    偕臧：

    刚看了一下欢迎，你的这个提的比较早，当时还没有 bot 还每天添加自动添加审查者功能（现在已经有了）， 所以没有人 code review； 这会已经添加对应的审查者了。



    这种长期没有回复的话，可以 @我

    https://github.com/linuxdeepin/deepin-draw/issues/9


    那这个是什么情况？都两周了，我直接close了

    Reply View the author
    juteman
    deepin
    2020-12-23 00:29
    #11
    chenzhiwei：

    借楼问下，我这个PR格式规范不？


    https://github.com/linuxdeepin/deepin-terminal/pull/48



    不规范

    Reply View the author
    chenzhiwei
    deepin
    2020-12-23 23:03
    #12
    juteman：

    不规范

    能说说具体是哪个地方吗？我是按照要求写的。



    Reply View the author
    juteman
    deepin
    2020-12-23 23:19
    #13
    chenzhiwei：

    能说说具体是哪个地方吗？我是按照要求写的。



    [汗]不好意思，看错了

    Reply View the author
    juteman
    deepin
    2020-12-23 23:29
    #14
    chenzhiwei：

    能说说具体是哪个地方吗？我是按照要求写的。



    你现在看你的 pull request ,应该有一些检查了，我叫南京那边抓紧审核一下

    Reply View the author