Comment 3 for bug 1912840

Revision history for this message
Jane Sandberg (sandbergja) wrote :

Thanks for the patch, Llewellyn, and for the testing, Garry. I was about to push it, but noticed:

1. The commit is missing your signoff, Llewellyn.
2. The commit doesn't include the LP number in the commit message.
3. There is still a console.log debugger line in the commit.

With your permission, Llewellyn, I can amend your commit to add the first two, and push a follow-up commit for the third item. Or, if you're up for amending this commit, that would also be great.