
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">

<html>

<head>

<meta http-equiv="Content-Type" content="text/xhtml; charset=utf-8">

</head>

<body>

<div style="font-family:sans-serif"><div style="white-space:normal">

<p dir="auto">I created a new PR, it has 2 commits, one to change the formatting and make it <code style="background-color:#F7F7F7; border-radius:3px; margin:0; padding:0 0.4em" bgcolor="#F7F7F7">long-line</code> style, which seems to be the dominant style in the code.</p>


<p dir="auto">Who is the maintainer ?</p>


<p dir="auto">On 2 Aug 2017, at 16:52, Thomas Sibley wrote:</p>


<blockquote style="border-left:2px solid #777; color:#777; margin:0 0 5px; padding-left:5px">

<p dir="auto">Functionally it looks sane to me, by quick inspection.  A few review notes on style and process:</p>


<p dir="auto">• At the very least, it should be reduced to the single real commit, instead of commit A, revert of A, commit B.</p>


<p dir="auto">• The maintainers may require you to add tests for this new option.</p>


<p dir="auto">• The maintainers would likely appreciate if you matched the surrounding code style, particularly with regard to indentation levels and brace hugging.</p>


<p dir="auto">Cheers,</p>


<p dir="auto">Thomas</p>

</blockquote>

</div>

</div>

</body>

</html>