• heluecht@pirati.ca
    link
    fedilink
    arrow-up
    2
    ·
    13 小时前

    @Petter1 @remington at our company every PR needs to be reviewed by at least one lead developer. And the PRs of the lead developers have to be reviewed by architects. And we encourage the other developers to perform reviews as well. Our company encourages the usage of Copilot. But none of our reviewers would pass code that they don’t understand.

      • heluecht@pirati.ca
        link
        fedilink
        arrow-up
        2
        ·
        1 小时前

        I’m a lead developer. And often I hear from my architect when I missed stuff in some PR that I just checked.

        I worked in a lot of different software companies over the last 35 years. And this company has by far the highest standards. It’s sometimes really annoying when you maybe coded 8 hours for a use case, just to spend 10-12 additional hours just for the test cases and may some 1-2 additional hours because the QA or the PO found something that needs to be changed. But in the end we can be proud of what we coded.