Introduce a new CommitAuthor class

This will be used in !86 (closed).

mentioned in merge request !86 (closed)

assigned to @rspeicher

@rymai I found it extremely difficult to follow the canonical_name method body, and I think it's a sign that we're bending over backwards to accommodate for a deficiency in the GitLab API. I think we should probably fix the problem there.

What do you think?

assigned to @rymai

I found it extremely difficult to follow the canonical_name method body

Yeah that's kind of a beast, I can try to improve it.

I think we should probably fix the problem there.

@rspeicher Yeah that sounds good, I will investigate that. In the meantime, that'd be great to move forward so that we can get !86 (closed) merged sometimes soon.

added 1 commit

• 00297d89 - Add httparty to the Gemfile and introduce TeamMember and Team classes

Compare with previous version

@jameslopez Would you like to review this?

assigned to @jameslopez

resolved all discussions

added 1 commit

• bfee62c5 - Add httparty to the Gemfile and introduce TeamMember and Team classes

Compare with previous version

Thanks @rymai! Looks really good. Left a few small comments. Also, the yml files you added to the specs are quite big, do they need to be that huge?

assigned to @rymai