diff --git a/docs/contributing.rst b/docs/contributing.rst index 34e92d8f0..f5c1c74b8 100644 --- a/docs/contributing.rst +++ b/docs/contributing.rst @@ -228,9 +228,10 @@ with a name of the branch you want to create locally). See also: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally#modifying-an-inactive-pull-request-locally. When writing GitHub pull requests, try to keep titles short but descriptive. -E.g. For bug #411: "Scrapy hangs if an exception raises in start" prefer -"Fix hanging when exception occurs in start (#411)" instead of "Fix for -#411". Complete titles make it easy to skim through the issue tracker. +E.g. For bug #411: "Scrapy hangs if an exception raises in start_requests" +prefer "Fix hanging when exception occurs in start_requests (#411)" +instead of "Fix for #411". Complete titles make it easy to skim through +the issue tracker. Finally, try to keep aesthetic changes (:pep:`8` compliance, unused imports removal, etc) in separate commits from functional changes. This will make pull