<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>