[11.0][MIG] mail_optional_autofollow

[damcar]

No description provided.

[JordiBForgeFlow]

You are missing one commit from v10. See f684fdf

Are you still actively working on this? Otherwise we can just merge from #251

[damcar]

@jbeficent @yajo it's ok for merged ?

[yajo]

Code OK.

Please squash all migration-related commits to merge.

[hbrunn]

as @jbeficent's comments seem to be fixed, I'll merge

[pedrobaeza]

Well, there's a lot of commit noise, so next time you can use your super-powers for squashing selectively them before merging.

[hbrunn]

is there a convenient way to do this without checking out locally, doing the rebase there and force pushing?

[pedrobaeza]

I'm afraid not. I do it locally, but I agree it's not the more comfortable thing to do. I have taught all my colleagues now to do forced push when changing PRs as GitHub has included the option to see the diff when you do this operation, so no extra action is required when merging.

[hbrunn]

hmmm, okay, thanks. So I'll keep asking people to squash themselves, I always dislike it when my signature is squashed away.

[pedrobaeza]

Yeah, that's another thing when manipulating commits... 😢

I think we can start to ask to forced push in the PRs instead.