Github usernames are case-insensitive

[ljharb]

enzymejs/enzyme#342 (comment) wasn't recognized because the capitalization on the username is different https://github.com/airbnb/enzyme/blob/master/MAINTAINERS#L4

Github usernames are (and should be) case-insensitive.

[ljharb]

I'm not familiar with go, but

lgtm/web/hook.go

Lines 112 to 116 in 66ef421

	if config.SelfApprovalOff && comment.Author == issue.Author {
	continue
	}
	// the user must be a valid maintainer of the project
	person, ok := maintainer.People[comment.Author]

implies to me that all usernames should be lowercased as soon as they're received from github, to avoid having to do it in a multitude of places?

[Idrinth]

at the very least it should be made clear in the documentation, that this doesn't follow the usual case-insensitive approach.

[ljharb]

At a minimum yes, but this is a pretty important inconsistency that should ideally be fixed :-/