← Back to context

Comment by olex

21 days ago

> Once Copilot is done, it’ll tag you for review. You can ask Copilot to make changes by leaving comments in the pull request.

To me, this reads like it'll be a good junior and open up a PR with its changes, letting you (the issue author) review and merge. Of course, you can just hit "merge" without looking at the changes, but then it's kinda on you when unreviewed stuff ends up in main.

A good junior has strong communication skills, humility, asks many good questions, has imagination, and a tremendous amount of human potential.

  • Has a point of view, a clear motive, ability to think holistically about things that are hard to digitize, get mad and clean up a bunch of stuff absolutely correctly because they're finally just "sick of all of this shit", or, conservatively isolates legacy code, studying it and creating buffering wrappers for the new system in pieces as the legacy issues are mitigated with a long term strategy. Each move is discussed with their peers. etc etc etc thank you for advocating sanity!

Management: "Why aren't you going faster now that the AI generates all the code and we fired half the dev team?"