Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
doc: adding thlorenz to list of collaborators
Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>, Jeremiah Senkpiel <fishrock123@rocketmail.com>, Bert Belder <bertbelder@gmail.com> PR-URL: #577
- Loading branch information
f3fed51
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, nits here, but something like this is the more typical commit formatting: 5ba307a
f3fed51
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, ok, I'll keep that in mind for next time, thanks.
f3fed51
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bnoordhuis are we ok with the reviewed-by list being on a single line here? Does it matter for any reason?
f3fed51
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rvagg et al, I would prefer one Reviewed-By line per reviewer. It makes it easier to parse from a script.
EDIT: Also, because of the 72 column limit.