I hope I got it right this time. (OT how can I get rid off those unnecessary commits when PR?)

Looks good to me!

how can I get rid off those unnecessary commits when PR?

git rebase -i origin/master is useful here -- it gives you the option to squash commits with their predecessor, so you can just change all but the first "pick" to "squash". if you've already pushed the branch, then you'll need to force push since this rewrites history.

but really if it's just a few commits, it's usually not necessary to squash them. i only get picky when there are a ton of commits, or they make and then undo unrelated changes (because it makes the version history confusing)

also: I'd recommend working in a branch git checkout -b <branchname> when you're putting together a pull request.