<p></p>
<p>Not so quick. While I value <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/users/joto/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/joto">@joto</a>'s opinions, he is not the one with the power over the merge button around here. ;)</p>
<p>It is true that we need to reconsider the approach to configuration options. But that is not going to happen today or next week and we need to be practical here. This is a feature that has been requested quite often and I'm on record stating that I take a pull request that implements this as an extra option. This is still true. On a quick look into it, this PR meets the requirements.</p>
<p>So I'll reopen the PR in the hope that we might get it in before the next release (which is rather imminent). I still need to do a detailed review later. For the moment, two of the points that <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/users/joto/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/joto">@joto</a> mentions, I'd definitely liked to see addressed, too: the documentation needs to be adapted and we should have at least one test. The test could consist of an additional regression test or do as <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/users/joto/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/joto">@joto</a> says. <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/users/zerebubuth/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/zerebubuth">@zerebubuth</a> I'd really appreciate if you can look into this.</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/osm2pgsql/pull/1241#issuecomment-653539199">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AA6353VR4SDAERAPMOBMF4TRZXJS7ANCNFSM4OPABPQQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AA6353X3OLU5M7D7B5FM5NLRZXJS7A5CNFSM4OPABPQ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOE32DO7Y.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/osm2pgsql/pull/1241#issuecomment-653539199",
"url": "https://github.com/openstreetmap/osm2pgsql/pull/1241#issuecomment-653539199",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>