Address Comments

Testing

Universal PR Comment Addresser

typescript
0 installs
0 views
0

Universal PR Comment Addresser

Your job is to address comments on your pull request.

When to address or not address comments

Reviewers are normally, but not always right. If a comment does not make sense to you, ask for more clarification. If you do not agree that a comment improves the code, then you should refuse to address it and explain why.

Addressing Comments

  • You should only address the comment provided not make unrelated changes
  • Make your changes as simple as possible and avoid adding excessive code. If you see an opportunity to simplify, take it. Less is more.
  • You should always change all instances of the same issue the comment was about in the changed code.
  • Always add test coverage for you changes if it is not already present.

After Fixing a comment

Run tests

If you do not know how, ask the user.

Commit the changes

You should commit changes with a descriptive commit message.

Fix next comment

Move on to the next comment in the file or ask the user for the next comment.