Moving 8.5 change to 8.4 since we're merging master into stable for now.

[ci skip]

[ci skip]

Improve the consistency of commit titles, branch names, tag names, issue/MR titles, on their respective project pages

Fix has been introduced in !2224.

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

[ci skip]

I forgot to add these in the respective MRs.

[ci skip]

This would help admins figure out from where spam is originating.

[ci skip]

Closes #4231

Closes #5851

Properly ensure that the token exists and add defensively check for a
non-nil value.

Closes #4294

[ci skip]

Closes #3268

Also, Open Graph data, if you're into that kind of thing.

[ci skip]

When deleting a branch, this is what was happening:

1. DeleteBranchService calls EventCreateService and creates an event.
2. The call to repository.rm_branch triggers the GitHooksService.
3. This, in turn, calls GitPushService and then calls EventCreateService again.

5145706c now makes it no longer necessary for DeleteBranchService and CreateBranchService to create an event.

Closes #4304

[ci skip]

Every time I push to GitLab, I get > 2 emails saying a spec failed when
I don't care about benchmarks and other specs that have `allow_failure` set to `true`.

Closes https://github.com/gitlabhq/gitlabhq/issues/9956

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>