<p>Thanks! I applied you commits manually with <code>git am ...</code> after editing the first line of each commit message to follow the contributing guidelines:</p>
<ul>
<li><a href="https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md#commit-message-format">https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md#commit-message-format</a></li>
</ul>
<p>Respectively, I added the <code>imc: </code> prefix there.</p>
<p>We use git log to generate the changelog for each release and it is useful to spot quickly which component was affected by that commit.</p>
<p>Another remark for the future: you do not need to generate the README locally and then create a commit for it. You can generate it locally to check if all is ok with the docbook syntax, but then you can delete the file and do git checkout path/to/the/README. This is because we generate the readme files for modules via a cron.d job on kamailio server in order to have coherent format of the output. In the past, a small change to the docbook file could end up in a completely different style of whitespacing in the README, depending on the system where the README was generated.</p>
<p>Last, you have git write privileges, so you can push commits directly without making pull requests. If you want someone to review the patches, then of course you can do pull requests (btw, I haven't really reviewd the code of your commits in this pull request at this moment, only edited the patch set for proper commit message format). The imc module doesn't have an active maintainer, you can step in if you plan to work more on this module, so there is no need for additional review unless you want explicitly.</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/kamailio/kamailio/pull/1869#issuecomment-467512272">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZXfW4MD2gHKZ0QfehEMAFP0L31D1ks5vRWJSgaJpZM4bSgUz">mute the thread</a>.<img src="https://github.com/notifications/beacon/AF36ZTCPmDyaUjkkzQZLli-1wx6A37Y_ks5vRWJSgaJpZM4bSgUz.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/kamailio/kamailio","title":"kamailio/kamailio","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/kamailio/kamailio"}},"updates":{"snippets":[{"icon":"PERSON","message":"@miconda in #1869: Thanks! I applied you commits manually with `git am ...` after editing the first line of each commit message to follow the contributing guidelines:\r\n\r\n  * https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md#commit-message-format\r\n\r\nRespectively, I added the `imc: ` prefix there.\r\n\r\nWe use git log to generate the changelog for each release and it is useful to spot quickly which component was affected by that commit.\r\n\r\nAnother remark for the future: you do not need to generate the README locally and then create a commit for it. You can generate it locally to check if all is ok with the docbook syntax, but then you can delete the file and do git checkout path/to/the/README. This is because we generate the readme files for modules via a cron.d job on kamailio server in order to have coherent format of the output. In the past, a small change to the docbook file could end up in a completely different style of whitespacing in the README, depending on the system where the README was generated.\r\n\r\nLast, you have git write privileges, so you can push commits directly without making pull requests. If you want someone to review the patches, then of course you can do pull requests (btw, I haven't really reviewd the code of your commits in this pull request at this moment, only edited the patch set for proper commit message format). The imc module doesn't have an active maintainer, you can step in if you plan to work more on this module, so there is no need for additional review unless you want explicitly."}],"action":{"name":"View Pull Request","url":"https://github.com/kamailio/kamailio/pull/1869#issuecomment-467512272"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/kamailio/kamailio/pull/1869#issuecomment-467512272",
"url": "https://github.com/kamailio/kamailio/pull/1869#issuecomment-467512272",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>