bors takes credit for squashed PRs

[camelid]

bors takes authorship credit for @bors squash PRs: rust-lang/rust@4b80687 (PR is rust-lang/rust#81398).

Also, not sure if this is related, but GitHub doesn't realize the PR is merged when bors squash-merges it (you can see an example in the same PR).

[camelid]

I believe this is the code at fault:

homu/homu/main.py

Lines 969 to 977 in 93ea32a

	args = [base_sha]
	if repo_cfg.get('autosquash', False):
	args += ['-i', '--autosquash']
	utils.logged_call(git_cmd('-c',
	'user.name=' + git_cfg['name'],
	'-c',
	'user.email=' + git_cfg['email'],
	'rebase',
	*args))

I think git_cfg is bors's config, not the PR author's.

[kellda]

@camelid autosquash is not actually used, the code at fault is a bit further down (implemented in 143278a)

Is to ok to take the info from the first or last commit ?

[kellda]

Btw it seems one can't tell GitHub that a PR was merged, and GitHub can't find it by itself since the merged commit is not the same as those originally in the PR.

[RalfJung]

Opened #181 for the PR closing part; let's not track two problems in one issue -- the title says this is about the commit author.