Message ID | 20210702201643.GA13765@locust (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [GIT,PULL] xfs: new code for 5.14 | expand |
On Fri, Jul 2, 2021 at 1:16 PM Darrick J. Wong <djwong@kernel.org> wrote: > > Please let me know if anything else strange happens during the merge > process. The merge commits I made seem stable enough, but as it's the > first time I've ever accepted a pull request, we'd all be open to > feedback for improvements for next time. It looks fine to me. I *would* suggest editing the merge commit messages a bit when doing pull requests from other people. It's by no means a big deal, but it looks a bit odd to see things like Hi all, ... Questions, comment and feedback appreciated! Thanks all! Allison in the merge message. All that text made a ton of sense in Allison's pull request, but as you actually then merge it, it doesn't make a lot of sense in the commit log, if you see what I mean.. But it's not a problem for this pull request, and I've merged it in my tree (pending my usual build tests etc, and I don't expect any issues). Linus
The pull request you sent on Fri, 2 Jul 2021 13:16:43 -0700:
> git://git.kernel.org/pub/scm/fs/xfs/xfs-linux.git tags/xfs-5.14-merge-6
has been merged into torvalds/linux.git:
https://git.kernel.org/torvalds/c/9f7b640f001f9781e0803fb60e7b3e7f2f1a1757
Thank you!
On Fri, Jul 02, 2021 at 02:39:46PM -0700, Linus Torvalds wrote: > On Fri, Jul 2, 2021 at 1:16 PM Darrick J. Wong <djwong@kernel.org> wrote: > > > > Please let me know if anything else strange happens during the merge > > process. The merge commits I made seem stable enough, but as it's the > > first time I've ever accepted a pull request, we'd all be open to > > feedback for improvements for next time. > > It looks fine to me. > > I *would* suggest editing the merge commit messages a bit when doing > pull requests from other people. > > It's by no means a big deal, but it looks a bit odd to see things like > > Hi all, > > ... > > Questions, comment and feedback appreciated! > > Thanks all! > Allison > > in the merge message. All that text made a ton of sense in Allison's > pull request, but as you actually then merge it, it doesn't make a lot > of sense in the commit log, if you see what I mean.. Yep, got it. I'll strip those out next time, thanks for helping me figure these things out. :) --D > > But it's not a problem for this pull request, and I've merged it in my > tree (pending my usual build tests etc, and I don't expect any > issues). > > Linus