Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ContentMine repositories lack .mailmap files #13

Open
ghost opened this issue Jun 5, 2017 · 3 comments
Open

ContentMine repositories lack .mailmap files #13

ghost opened this issue Jun 5, 2017 · 3 comments

Comments

@ghost
Copy link

ghost commented Jun 5, 2017

For several ContentMine repositories, running git shortlog -nes gives messy output, because contributors have used many aliases and there is no .mailmap to reconcile them. This is a documentation problem, making it hard to tell who has contributed to which repositories.

I submitted a PR against Norma in relation to this, but really it ought to be fixed across multiple repositories.

This is readily solvable by creating a repository with a .mailmap suitable for all ContentMine repositories, so that maintainers of each other ContentMine repository can include it as a Git submodule (or, it they prefer, a Git subtree).

@ghost ghost changed the title No .mailmap files ContentMine repositories lack .mailmap files Jun 5, 2017
@petermr petermr closed this as completed Jun 15, 2017
@ghost
Copy link
Author

ghost commented Jun 15, 2017

Re-opening, as issue has not been resolved and seems to have been closed erroneously. (Cf. #5 (comment) .)

@ghost
Copy link
Author

ghost commented Jun 19, 2017

My ContentMine consultancy is over, but I still have some local, unmerged work relating to this issue. To tie up those loose ends, I will submit some PRs shortly that should fix this issue within the AMI stack, and which can be merged by ContentMine staff whenever they wish.

ContentMine staff are of course welcome to apply the same patches to other ContentMine repositories.

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

No branches or pull requests

1 participant