I updated and now the codes is available for hiops module.<br>
In Regards of new function in core, I will do that.<br>
<br>
<br>
On Wed, Feb 20, 2019 at 2:15 PM Daniel-Constantin Mierla <<br>
notifications@github.com> wrote:<br>
<br>
> There is no code for your module, it seems you try to add it as a git<br>
> submodule. It has to be pushed with full code, if you want it to merge it<br>
> inside kamailio repository.<br>
><br>
> Then your commit messages do not follow the contributing guidelines:<br>
><br>
> -<br>
> https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md#commit-message-format<br>
><br>
> I also suggest that you put the function you added in core to your module,<br>
> it seems to target usage only there and there is no reason it doesn't work<br>
> having it inside your module. If it needs to be used in other modules<br>
> later, it can be relocated.<br>
><br>
> —<br>
> You are receiving this because you are subscribed to this thread.<br>
> Reply to this email directly, view it on GitHub<br>
> <https://github.com/kamailio/kamailio/pull/1856#issuecomment-465522509>,<br>
> or mute the thread<br>
> <https://github.com/notifications/unsubscribe-auth/ArE2rGb76fmIyAZqmVh6E3u57X8b4_nAks5vPScsgaJpZM4a-_NY><br>
> .<br>
><br>
<br>
<br>
-- <br>
--Mojtaba Esfandiari.S<br>
<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/1856#issuecomment-465940688">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AF36ZTrm5LKYo6iPY0HcTAj2Rht8wWsjks5vPnBxgaJpZM4a-_NY">mute the thread</a>.<img src="https://github.com/notifications/beacon/AF36ZRF6mMu3LM8OXeHr9GY-4Vtuf7WHks5vPnBxgaJpZM4a-_NY.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":"@mojtabaesfandiari in #1856: I updated and now the codes is available for hiops module.\nIn Regards of new function in core, I will do that.\n\n\nOn Wed, Feb 20, 2019 at 2:15 PM Daniel-Constantin Mierla \u003c\nnotifications@github.com\u003e wrote:\n\n\u003e There is no code for your module, it seems you try to add it as a git\n\u003e submodule. It has to be pushed with full code, if you want it to merge it\n\u003e inside kamailio repository.\n\u003e\n\u003e Then your commit messages do not follow the contributing guidelines:\n\u003e\n\u003e -\n\u003e https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md#commit-message-format\n\u003e\n\u003e I also suggest that you put the function you added in core to your module,\n\u003e it seems to target usage only there and there is no reason it doesn't work\n\u003e having it inside your module. If it needs to be used in other modules\n\u003e later, it can be relocated.\n\u003e\n\u003e —\n\u003e You are receiving this because you are subscribed to this thread.\n\u003e Reply to this email directly, view it on GitHub\n\u003e \u003chttps://github.com/kamailio/kamailio/pull/1856#issuecomment-465522509\u003e,\n\u003e or mute the thread\n\u003e \u003chttps://github.com/notifications/unsubscribe-auth/ArE2rGb76fmIyAZqmVh6E3u57X8b4_nAks5vPScsgaJpZM4a-_NY\u003e\n\u003e .\n\u003e\n\n\n-- \n--Mojtaba Esfandiari.S\n"}],"action":{"name":"View Pull Request","url":"https://github.com/kamailio/kamailio/pull/1856#issuecomment-465940688"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/kamailio/kamailio/pull/1856#issuecomment-465940688",
"url": "https://github.com/kamailio/kamailio/pull/1856#issuecomment-465940688",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>