
<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div>+1 to proposal and squashing prior. As well this process should be clearly documented for future contrib/ things.<br><br>Sent from my iPhone</div><div><br>On Jul 16, 2015, at 11:56, Simone Dalmasso <<a href="mailto:simone.dalmasso@gmail.com">simone.dalmasso@gmail.com</a>> wrote:<br><br></div><blockquote type="cite"><div><div dir="ltr">Thanks Jeff for bringing this up. <div>It is amazing to see the activity in contrib, this is just awesome but the more it grows the more there's the risk to loose the track on what's in.</div><div>my +1 on everything and I should add, especially for @simod :)<div><br></div><div>* the commits should be squashed to 1 </div><div><br></div><div>Ciao</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-07-16 17:48 GMT+02:00 Jeffrey Johnson <span dir="ltr"><<a href="mailto:ortelius@gmail.com" target="_blank">ortelius@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi All,<br>

<br>

I just pushed a commit directly to master that broke the build.<br>

<a href="https://github.com/GeoNode/geonode/commit/4baca0a24b65b9a2c656c47fd54c8c2f1c794acb" rel="noreferrer" target="_blank">https://github.com/GeoNode/geonode/commit/4baca0a24b65b9a2c656c47fd54c8c2f1c794acb</a><br>

<br>

This puts the geonode.contrib apps in GEONODE_CONTRIB_APPS in<br>

settings.py and makes sure they are *always* disabled by default.<br>

<br>

The test failure is here<br>

<a href="https://travis-ci.org/GeoNode/geonode/builds/71270871#L1645" rel="noreferrer" target="_blank">https://travis-ci.org/GeoNode/geonode/builds/71270871#L1645</a><br>

<br>

And the root cause is here<br>

<br>

<a href="https://github.com/GeoNode/geonode/pull/2206/files#diff-33cf4fcb0f4b5b8bf97b2d580c7eaec1R81" rel="noreferrer" target="_blank">https://github.com/GeoNode/geonode/pull/2206/files#diff-33cf4fcb0f4b5b8bf97b2d580c7eaec1R81</a><br>

<br>

No need to point fingers as this PR went through the process we have<br>

setup and was merged. We do however need to have some more guidelines<br>

about how contrib modules should be handled.<br>

<br>

After a quick chat with Simone and Ariel, I propose the following:<br>

<br>

* New contrib modules should be initially proposed as a GNIP and<br>

brought up on the list before submitting a PR<br>

* Contrib modules should go into the GEONODE_CONTRIB_APPS block in<br>

settings and _never_ be enabled by default.<br>

* Contrib modules should have their own requirements.txt and not add<br>

things to setup.py<br>

* Contrib modules should be verified to not break the build when<br>

enabled/disabled before merging<br>

* There should be minimal docs about what it does and how to use it<br>

etc. A README in the module is sufficient, a page in docs/<br>

<br>

Anything else I missed? Im +1 to keep the party going and looking<br>

forward to seeing interesting things in this part of the codebase, but<br>

we cant get sloppy.<br>

<br>

Jeff<br>

_______________________________________________<br>

geonode-devel mailing list<br>

<a href="mailto:geonode-devel@lists.osgeo.org">geonode-devel@lists.osgeo.org</a><br>

<a href="http://lists.osgeo.org/cgi-bin/mailman/listinfo/geonode-devel" rel="noreferrer" target="_blank">http://lists.osgeo.org/cgi-bin/mailman/listinfo/geonode-devel</a><br>

</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Simone </div>

</div>

</div></blockquote><blockquote type="cite"><div><span>_______________________________________________</span><br><span>geonode-devel mailing list</span><br><span><a href="mailto:geonode-devel@lists.osgeo.org">geonode-devel@lists.osgeo.org</a></span><br><span><a href="http://lists.osgeo.org/cgi-bin/mailman/listinfo/geonode-devel">http://lists.osgeo.org/cgi-bin/mailman/listinfo/geonode-devel</a></span><br></div></blockquote></body></html>