On Mon, 2023-05-15 at 15:33 -0300, Andrew Wetmore wrote: > Hi, Rich. > > I was afk so was slow to review your pull request. I wanted to > suggest some > minor changes, but by the time I submitted my comments the pull > request had > happened. > > My comments were: > > line 7: "The addition of new committers" -- "new" is not necessary
Ok. I suppose I could see that. > line 17: "inviting a new committer" -- "new" is not necessary The whole emphasis of this doc is new, not existing, committers. Inviting a committer implies, to me, an existing one, and I think "new" clarifies here, even if it is not strictly necessary. > line 34: "committers that" should be "committers who" > line 49: I think "whims" should be "whim" > line 50: "someone that" should be "someone who" > line 74: need a comma after the link and before "which" > lines 77-78: "that a committers" should be either "that a committer" > or > "that committers" All changes made. Thanks. --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@community.apache.org For additional commands, e-mail: dev-h...@community.apache.org
