doc: remove tentativeness in pull-requests.md

There are some uses of "in general" that are unnecessary. (We don't need to be tentative about the fact that tests should pass, for example.)

Checklist

• documentation is changed or added
• commit message follows commit guidelines

Affected core subsystem(s)

doc