What merging methods to use?

[EvitanRelta]

Context

So far, the merging method I've used is:

• ensure branch is up-to-date (if not, rebase branch onto master)
• merge by git merge --no-ff [branch]
• no squash
• then git commit --am, and add an issue-closing line to the merge-commit's body (eg. Fixes #4, Closes #5)

Which gives a history that looks like this:

The question

I'm still fairly new in the programming world, so idk what is the industry standard for this.

• To squash?
• To rebase instead of merge?
• Which commit to put the issue-closing keywords (eg. Fixes #3) on? The merge commit? Or the commits in the branch?

If anyone with some experience in this, and knows the pros/cons of each, please advise! D: