<font size=2 face="sans-serif">Thai -</font>
<br>
<br><font size=2 face="sans-serif">I thought there was a conscious decision
to separate addressing the JSCS issues and applying JP's guidelines.  
 Smaller more precise patches are easier to land!</font>
<br><font size=2 face="sans-serif">I plan on applying JP's guidelines in
a follow on patch.    Especially since one of my JSCS patches
merged this morning.</font>
<br><font size=2 face="sans-serif"><br>
<br>
Aaron D. Sahlin<br>
IBMUSM07(asahlin)<br>
Dept. X2WA<br>
Phone 507-253-7349 Tie 553-7349<br>
</font>
<br>
<br>
<br>
<br><font size=1 color=#5f5f5f face="sans-serif">From:      
 </font><font size=1 face="sans-serif">Thai Q Tran/Silicon
Valley/IBM@IBMUS</font>
<br><font size=1 color=#5f5f5f face="sans-serif">To:      
 </font><font size=1 face="sans-serif">"OpenStack Development
Mailing List \(not for usage questions\)" <openstack-dev@lists.openstack.org></font>
<br><font size=1 color=#5f5f5f face="sans-serif">Date:      
 </font><font size=1 face="sans-serif">06/04/2015 01:33 PM</font>
<br><font size=1 color=#5f5f5f face="sans-serif">Subject:    
   </font><font size=1 face="sans-serif">[openstack-dev]
[horizon] Prioritize tests over JSCS</font>
<br>
<hr noshade>
<br>
<br>
<br><font size=2 face="sans-serif">Hi folks,</font>
<br>
<br><font size=2 face="sans-serif">I know a lot of people are tackling
the JSCS stuff, and thats really great. But it would be extra nice to see
JSCS stuff along with JP's guidelines in your patches. Furthermore, if
the file you are working on doesn't have an accompanying spec file, please
make sure that the tests for it exists. If it is not there, please prioritize
and spend some time reviewing patches with the tests you need, or create
a spec file and get that merge first.</font>
<br>
<br><font size=2 face="sans-serif">Thanks</font><tt><font size=2>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: OpenStack-dev-request@lists.openstack.org?subject:unsubscribe<br>
</font></tt><a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev"><tt><font size=2>http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</font></tt></a><tt><font size=2><br>
</font></tt>
<br>