# Extension contribution checklist

**URL:** <https://discourse.slicer.org/t/extension-contribution-checklist/4975>\
**Category:** Development\
**Created:** [December 5, 2018, 6:18pm UTC](https://discourse.slicer.org/t/extension-contribution-checklist/4975 "2018-12-05T18:18:03Z")\
**Posts on this page:** 4\
**Page:** 1

<div class="post-metadata">

**Author:** ![jcfr](https://sea2.discourse-cdn.com/flex002/user_avatar/discourse.slicer.org/jcfr/32/17825_2.png) [@jcfr](https://discourse.slicer.org/u/jcfr)\
**Post date:** [December 5, 2018, 6:18pm UTC](https://discourse.slicer.org/t/extension-contribution-checklist/4975/1 "2018-12-05T18:18:04Z")

</div>

### extension repository name

Earlier today, following the submission of a [pull request](https://github.com/Slicer/ExtensionsIndex/pull/1597) adding a new extension to the Extension index, the associated test failed:

 ![image](https://us1.discourse-cdn.com/flex002/uploads/slicer/original/3X/1/5/15a4b0bc0bbe4277a97d5a58db575c23ea3dc394.png)

with the following message:

 ![image](https://us1.discourse-cdn.com/flex002/uploads/slicer/original/3X/e/7/e7d12506f40ee4f491e71cbb8f20c31a7ff1f332.png) .

Following [input](https://github.com/Slicer/ExtensionsIndex/pull/1597#issuecomment-444535467) from @fedorov , we tweaked the wording to mention that using the `Slicer` prefix was recommended but not mandatory. See [update](https://github.com/Slicer/ExtensionsIndex/pull/1600) to the ci script and wiki.

### contribution checklist improvement

The checklist documented in the [Step-by-step: How to create, publish, distribute and maintain an extension ?](https://www.slicer.org/wiki/Documentation/Nightly/Developers/Tutorials/BuildTestPackageDistributeExtensions#Contribution_Checklist) guide has also been updated and is now in markdown format ready to be copied into new pull request. This should help extension index reviewer.

 ![image](https://us1.discourse-cdn.com/flex002/uploads/slicer/original/3X/9/f/9f8336708a5c18f02d1bc605ba8d900a6ae0633b.png)

---

<div class="post-metadata">

**Author:** ![brhoom](https://sea2.discourse-cdn.com/flex002/user_avatar/discourse.slicer.org/brhoom/32/1228_2.png) [@brhoom](https://discourse.slicer.org/u/brhoom)\
**Post date:** [December 6, 2018, 5:59pm UTC](https://discourse.slicer.org/t/extension-contribution-checklist/4975/2 "2018-12-06T17:59:25Z")

</div>

Thanks for your effort.

I don’t know why [my modules tests always fail](http://slicer.cdash.org/index.php?project=Slicer4&filtercount=1&showfilters=1&field1=buildname&compare1=63&value1=SlicerCochlea)?

The module download elastix binaries from our server and use them in its logic class. Is there anything I should to make things work? I run the testing in differentWindow and Ubuntu machines and it works. The extension does not support Mac for now but mac users are welcome to modify it and I can help in this if needed (I don’t have mac).

---

<div class="post-metadata">

**Author:** ![lassoan](https://sea2.discourse-cdn.com/flex002/user_avatar/discourse.slicer.org/lassoan/32/13_2.png) [@lassoan](https://discourse.slicer.org/u/lassoan)\
**Post date:** [December 6, 2018, 11:14pm UTC](https://discourse.slicer.org/t/extension-contribution-checklist/4975/3 "2018-12-06T23:14:48Z")

</div>

If you need Elastix, please do not download binaries from your own server. Instead, add SlicerElastix to EXTENSION\_DEPENDS in your CMakeLists.txt file. When you install your extension, the extension manager will also install SlicerElastix extension, which has all the Elastix binaries on all platforms (including MacOS) and compatible with Slicer (uses Slicer’s ITK). Using the proper binaries might also fix your module tests.

---

<div class="post-metadata">

**Author:** ![brhoom](https://sea2.discourse-cdn.com/flex002/user_avatar/discourse.slicer.org/brhoom/32/1228_2.png) [@brhoom](https://discourse.slicer.org/u/brhoom)\
**Post date:** [December 7, 2018, 7:54am UTC](https://discourse.slicer.org/t/extension-contribution-checklist/4975/4 "2018-12-07T07:54:39Z")

</div>

Thanks for the info. I will try that.
