<div dir="ltr"><div><div><div>Speaking of overzealous rules, The Flow rubocop job was failing on its Gemfile:<br><pre class=""><span class=""><b>02:53:31</b> </span>Gemfile:1:1: C: Missing space after #.
<span class=""><b>02:53:31</b> </span>#ruby=ruby-2.1.1
<span class=""><b>02:53:31</b> </span>^^^^^^^^^^^^^^^^
<span class=""><b>02:53:31</b> </span>Gemfile:2:1: C: Missing space after #.
<span class=""><b>02:53:31</b> </span>#ruby-gemset=Flow
<span class=""><b>02:53:31</b> </span>^^^^^^^^^^^^^^^^^</pre></div>but all our Gemfiles are written this way.<br><br></div>I stole the fix from CirrusSearch/.rubocop.yml :<br><br><span style="font-family:courier new,monospace">Style/LeadingCommentSpace:<br>  Exclude:<br>    - Gemfile  # RVM doesn't recognise spaces after the #s<br></span></div><div>and now the mwext-Flow-bundle-rubocop CI job passes.<br><br></div>Is there a way MediaWiki extensions can share common configuration and dev_scripts like this? Maybe extensions/shared, or extensions/Mantle/defaults<br><div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 31, 2014 at 4:05 PM, Dan Duvall <span dir="ltr"><<a href="mailto:dduvall@wikimedia.org" target="_blank">dduvall@wikimedia.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hey all,<br><div class="gmail_extra"><div><br></div><div>Zeljko, Antoine, and I have been working on a first draft of the Ruby coding conventions for MW-related projects, and dialing in the RuboCop checker accordingly.[1] I encourage anyone who has a bit of Ruby knowledge and/or an opinion on style to contribute to the conversion.[2]</div><div><br></div><div>As the article suggests, there are a handful of contentious rules in the bbatsov style guide from which the conventions are based. This first draft attempts to resolve those conflicts—mainly by ignoring the rules—and is intended to be organically revised as the discussion evolves—that kinda goes without saying given it's a wiki. :)</div><div><br></div><div>Just some clarification on the RuboCop check: 1) it's non-voting for the time being to give maintainers a chance to customize the configuration according to project/team needs; and 2) it should be green to start with since the configuration was auto-generated and ignores everything it finds by default.[3]</div><div><br></div><div>Let us know if you have any more questions about the checker. Happy arguing! (I mean that in all seriousness as I enjoy discussing semantics.)</div><div><br></div><div>Cheers,</div><div>Dan</div><div><br></div><div>[1] <a href="https://www.mediawiki.org/wiki/Manual:Coding_conventions/Ruby" target="_blank">https://www.mediawiki.org/wiki/Manual:Coding_conventions/Ruby</a></div><div>[2] <a href="https://www.mediawiki.org/wiki/Manual_talk:Coding_conventions/Ruby" target="_blank">https://www.mediawiki.org/wiki/Manual_talk:Coding_conventions/Ruby</a></div><div>[3] <a href="https://www.mediawiki.org/wiki/Manual:Coding_conventions/Ruby#RuboCop" target="_blank">https://www.mediawiki.org/wiki/Manual:Coding_conventions/Ruby#RuboCop</a></div><span class=""><font color="#888888"><div><br></div>-- <br><div><div dir="ltr">Dan Duvall<div>Automation Engineer</div><div><a href="http://wikimediafoundation.org" target="_blank">Wikimedia Foundation</a><br></div></div></div>
</font></span></div></div>
<br>_______________________________________________<br>
QA mailing list<br>
<a href="mailto:QA@lists.wikimedia.org">QA@lists.wikimedia.org</a><br>
<a href="https://lists.wikimedia.org/mailman/listinfo/qa" target="_blank">https://lists.wikimedia.org/mailman/listinfo/qa</a><br>
<br></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr">=S Page  Features engineer<br></div></div>
</div></div>