<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/gitlab/gitlab-ce.git/lib/declarative_policy, branch use-queue_namespace</title>
<subtitle>gitlab.com: gitlab-org/gitlab-ce.git
</subtitle>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/'/>
<entry>
<title>Adds Rubocop rule for line break after guard clause</title>
<updated>2017-11-16T16:58:29+00:00</updated>
<author>
<name>Jacopo</name>
<email>beschi.jacopo@gmail.com</email>
</author>
<published>2017-11-14T09:02:39+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=181cd299f9e06223e8338e93b1c318c671ccb1aa'/>
<id>181cd299f9e06223e8338e93b1c318c671ccb1aa</id>
<content type='text'>
Adds a rubocop rule (with autocorrect) to ensure line break after guard clauses.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Adds a rubocop rule (with autocorrect) to ensure line break after guard clauses.
</pre>
</div>
</content>
</entry>
<entry>
<title>Speed up cached_pass? for composite rules</title>
<updated>2017-10-05T09:12:58+00:00</updated>
<author>
<name>Sean McGivern</name>
<email>sean@gitlab.com</email>
</author>
<published>2017-10-04T13:13:22+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=e9476eb97af59404438d8403968a282d6d08a20d'/>
<id>e9476eb97af59404438d8403968a282d6d08a20d</id>
<content type='text'>
Both `Or` and `And` would evaluate whether each rule passed, then calculate a
value based on the results of all of those. We can actually return early in many
cases, without running the rule at all.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Both `Or` and `And` would evaluate whether each rule passed, then calculate a
value based on the results of all of those. We can actually return early in many
cases, without running the rule at all.
</pre>
</div>
</content>
</entry>
<entry>
<title>Speed up DeclarativePolicy::Runner#steps_by_score</title>
<updated>2017-10-04T12:50:05+00:00</updated>
<author>
<name>Sean McGivern</name>
<email>sean@gitlab.com</email>
</author>
<published>2017-10-04T12:43:39+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=d7717192782232c14c4f16962ab450b08348a4c9'/>
<id>d7717192782232c14c4f16962ab450b08348a4c9</id>
<content type='text'>
There were a couple of things here:

1. If the state was already enabled, we don't need to check all the remaining
   steps - only those that can prevent the state. (An enable followed by an
   enable is a no-op.) This logic is in `#run`, but we still did the work of
   scoring and sorting the steps.
2. The sorting is known to be inefficient, but we can make it slightly more
   efficient by stopping once we have a step with zero score, as that means it's
   free.

Neither of these make this _fast_, especially when called lots of times - as we
do when there is lots of activity on an issue - but they do help some.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
There were a couple of things here:

1. If the state was already enabled, we don't need to check all the remaining
   steps - only those that can prevent the state. (An enable followed by an
   enable is a no-op.) This logic is in `#run`, but we still did the work of
   scoring and sorting the steps.
2. The sorting is known to be inefficient, but we can make it slightly more
   efficient by stopping once we have a step with zero score, as that means it's
   free.

Neither of these make this _fast_, especially when called lots of times - as we
do when there is lots of activity on an issue - but they do help some.
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge branch 'rs-more-public-send-whitelists' into 'master'</title>
<updated>2017-08-16T11:25:26+00:00</updated>
<author>
<name>Rémy Coutable</name>
<email>remy@rymai.me</email>
</author>
<published>2017-08-16T11:25:26+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=fcce6c3168dbd04404f318b4e983d7395d21538c'/>
<id>fcce6c3168dbd04404f318b4e983d7395d21538c</id>
<content type='text'>
Whitelist or fix additional `Gitlab/PublicSend` cop violations

See merge request !13467</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Whitelist or fix additional `Gitlab/PublicSend` cop violations

See merge request !13467</pre>
</div>
</content>
</entry>
<entry>
<title>Enable Layout/TrailingWhitespace cop and auto-correct offenses</title>
<updated>2017-08-15T17:44:37+00:00</updated>
<author>
<name>Robert Speicher</name>
<email>rspeicher@gmail.com</email>
</author>
<published>2017-08-15T17:44:37+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=4edfad96784e8f77ec8ead26f01b4012977ba58a'/>
<id>4edfad96784e8f77ec8ead26f01b4012977ba58a</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Whitelist or fix additional `Gitlab/PublicSend` cop violations</title>
<updated>2017-08-14T16:14:11+00:00</updated>
<author>
<name>Robert Speicher</name>
<email>rspeicher@gmail.com</email>
</author>
<published>2017-08-10T16:39:26+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=260c8da060a6039cbd47cfe31c8ec6d6f9b43de0'/>
<id>260c8da060a6039cbd47cfe31c8ec6d6f9b43de0</id>
<content type='text'>
An upcoming update to rubocop-gitlab-security added additional
violations.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
An upcoming update to rubocop-gitlab-security added additional
violations.
</pre>
</div>
</content>
</entry>
<entry>
<title>more eagerly bail when the state is prevented</title>
<updated>2017-08-07T19:48:22+00:00</updated>
<author>
<name>http://jneen.net/</name>
<email>jneen@jneen.net</email>
</author>
<published>2017-08-07T19:48:22+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=15bb6e1ed7a4fb1d6554e193497836b79bc26bda'/>
<id>15bb6e1ed7a4fb1d6554e193497836b79bc26bda</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>reduce iterations by keeping a count of remaining enablers</title>
<updated>2017-08-07T19:36:13+00:00</updated>
<author>
<name>http://jneen.net/</name>
<email>jneen@jneen.net</email>
</author>
<published>2017-08-07T19:36:13+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=8a167554a9dae4ab10f5a56124a598227bf5e3ed'/>
<id>8a167554a9dae4ab10f5a56124a598227bf5e3ed</id>
<content type='text'>
rather than iterating the whole remaining step set with
.all?(&amp;:prevent?)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
rather than iterating the whole remaining step set with
.all?(&amp;:prevent?)
</pre>
</div>
</content>
</entry>
<entry>
<title>cache the cache key...</title>
<updated>2017-07-17T21:38:48+00:00</updated>
<author>
<name>http://jneen.net/</name>
<email>jneen@jneen.net</email>
</author>
<published>2017-07-17T17:47:27+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=fcd3e5d4777efb4b6db2de08347e4db603e1dea2'/>
<id>fcd3e5d4777efb4b6db2de08347e4db603e1dea2</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>avoid #respond_to? in Cache.id_for</title>
<updated>2017-07-17T21:38:48+00:00</updated>
<author>
<name>http://jneen.net/</name>
<email>jneen@jneen.net</email>
</author>
<published>2017-07-17T17:43:55+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/gitlab/gitlab-ce.git/commit/?id=45e9d2879122d2aea9494241fac0bd8bf7096446'/>
<id>45e9d2879122d2aea9494241fac0bd8bf7096446</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
</feed>
