Skip to content

Comments

Improved .mailmap#443

Open
dstansby wants to merge 1 commit intoome:mainfrom
dstansby:clean-mailmap
Open

Improved .mailmap#443
dstansby wants to merge 1 commit intoome:mainfrom
dstansby:clean-mailmap

Conversation

@dstansby
Copy link
Contributor

Inspired by #442, this adds an improved .mailmap file to normalise email addresses and names used by git. Where multiple emails are available in the log, I've chosen the most used or the non-GitHub one.

cc @bugraoezdemir, @cfusterbarcelo, @clbarnes, @constantinpape, @giovp, @bogovicj, @jwindhager, @joshmoore, @kbab, @mkitti, @thewtex, @lubianat, @vuhlmann as folks I've updated here - let me know if you have a different preferred email or name.

mkitti

This comment was marked as resolved.

@thewtex
Copy link
Contributor

thewtex commented Feb 19, 2026

@dstansby thanks 👍

Mark Kittisopikul <kittisopikulm@janelia.hhmi.org> <markkitt@gmail.com>
Mark Kittisopikul <kittisopikulm@janelia.hhmi.org> <mkitti@users.noreply.github.com>
Matt McCormick <matt@fideus.io> <matt@mmmccormick.com>
Matt McCormick <matt@fideus.io> <matt@mmmccormick.com>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These two lines look identical, is that intended?

Josh Moore <j.a.moore@dundee.ac.uk>
Josh Moore <josh@glencoesoftware.com>
Josh Moore <josh@openmicroscopy.org>
Tiago Lubiana <tiago.lubiana.alves@usp.br> <tiagolubiana@gmail.com>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Would you mind replacing that by:

Tiago Lubiana <tiago.lubiana@gerbi-gmb.de> <tiagolubiana@gmail.com>
Tiago Lubiana <tiago.lubiana@gerbi-gmb.de> <tiago.lubiana.alves@usp.br>

@lubianat
Copy link
Contributor

@dstansby do you think you will have time to update this? I can take it over if you are busy.

We are doing a PR review in ~ 1 week (German-BioImaging/incubator#54) and would be great to get it then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants