<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/bundler.git/spec, branch colby/git-https-default</title>
<subtitle>github.com: bundler/bundler.git
</subtitle>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/bundler.git/'/>
<entry>
<title>add feature flag to use https for :git in the dsl</title>
<updated>2018-11-24T20:56:11+00:00</updated>
<author>
<name>Colby Swandale</name>
<email>me@colby.fyi</email>
</author>
<published>2018-11-22T04:43:23+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/bundler.git/commit/?id=93471b02962380f46271b832b5344760e532131d'/>
<id>93471b02962380f46271b832b5344760e532131d</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Followed up https://github.com/bundler/bundler/pull/6743</title>
<updated>2018-11-16T13:17:58+00:00</updated>
<author>
<name>SHIBATA Hiroshi</name>
<email>hsbt@ruby-lang.org</email>
</author>
<published>2018-11-16T13:17:58+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/bundler.git/commit/?id=66f5dcbd06edebc442dea6b01d0e8bc7449d1d54'/>
<id>66f5dcbd06edebc442dea6b01d0e8bc7449d1d54</id>
<content type='text'>
  #6743 was failed with ruby core repository and ignored working example.
  Fixed them.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
  #6743 was failed with ruby core repository and ignored working example.
  Fixed them.
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge #6775</title>
<updated>2018-11-16T10:37:20+00:00</updated>
<author>
<name>Bundlerbot</name>
<email>bot@bundler.io</email>
</author>
<published>2018-11-16T10:37:20+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/bundler.git/commit/?id=05b552ccafb6fc6492e498edaeddd0979657c517'/>
<id>05b552ccafb6fc6492e498edaeddd0979657c517</id>
<content type='text'>
6775: [Plugin::Index] Only register each plugin once for a given hook r=colby-swandale a=segiddins

### What was the end-user problem that led to this PR?

The problem was running `plugin install` twice for a plugin withs hooks would cause that hook to be registered twice.

Closes #6771.

### What was your diagnosis of the problem?

My diagnosis was a plugin's hooks should only be run once per event.

### What is your fix for the problem, implemented in this PR?

My fix is to `uniq` the list of plugins registered for each event.

Co-authored-by: Samuel Giddins &lt;segiddins@segiddins.me&gt;
Co-authored-by: Olle Jonsson &lt;olle.jonsson@gmail.com&gt;
Co-authored-by: Colby Swandale &lt;me@colby.fyi&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
6775: [Plugin::Index] Only register each plugin once for a given hook r=colby-swandale a=segiddins

### What was the end-user problem that led to this PR?

The problem was running `plugin install` twice for a plugin withs hooks would cause that hook to be registered twice.

Closes #6771.

### What was your diagnosis of the problem?

My diagnosis was a plugin's hooks should only be run once per event.

### What is your fix for the problem, implemented in this PR?

My fix is to `uniq` the list of plugins registered for each event.

Co-authored-by: Samuel Giddins &lt;segiddins@segiddins.me&gt;
Co-authored-by: Olle Jonsson &lt;olle.jonsson@gmail.com&gt;
Co-authored-by: Colby Swandale &lt;me@colby.fyi&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>fix rubocop</title>
<updated>2018-11-16T10:00:19+00:00</updated>
<author>
<name>Colby Swandale</name>
<email>me@colby.fyi</email>
</author>
<published>2018-11-16T10:00:19+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/bundler.git/commit/?id=19237b490b690b8f9cf922f2ac982ef684c23cee'/>
<id>19237b490b690b8f9cf922f2ac982ef684c23cee</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge #6743</title>
<updated>2018-11-16T02:44:32+00:00</updated>
<author>
<name>Bundlerbot</name>
<email>bot@bundler.io</email>
</author>
<published>2018-11-16T02:44:32+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/bundler.git/commit/?id=172ec59db53ec05a91c0d9eb72e2d715ae82f3c0'/>
<id>172ec59db53ec05a91c0d9eb72e2d715ae82f3c0</id>
<content type='text'>
6743: Support file structure of ruby core repository. r=hsbt a=hsbt

### What was the end-user problem that led to this PR?

In the ruby core repository, I put bundler executable and bundler libraries under the `bin` and `lib` directories. It breaks the current behavior.

### What is your fix for the problem, implemented in this PR?

Support the structure of ruby core repository.



Co-authored-by: SHIBATA Hiroshi &lt;hsbt@ruby-lang.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
6743: Support file structure of ruby core repository. r=hsbt a=hsbt

### What was the end-user problem that led to this PR?

In the ruby core repository, I put bundler executable and bundler libraries under the `bin` and `lib` directories. It breaks the current behavior.

### What is your fix for the problem, implemented in this PR?

Support the structure of ruby core repository.



Co-authored-by: SHIBATA Hiroshi &lt;hsbt@ruby-lang.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Double-register hook in spec</title>
<updated>2018-11-11T07:10:54+00:00</updated>
<author>
<name>Olle Jonsson</name>
<email>olle.jonsson@gmail.com</email>
</author>
<published>2018-11-11T07:10:54+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/bundler.git/commit/?id=2bf93186c42a1a13e822fe1fedcc811ec210b87b'/>
<id>2bf93186c42a1a13e822fe1fedcc811ec210b87b</id>
<content type='text'>

To make it clear the list is uniq’d

Co-Authored-By: segiddins &lt;segiddins@segiddins.me&gt;</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>

To make it clear the list is uniq’d

Co-Authored-By: segiddins &lt;segiddins@segiddins.me&gt;</pre>
</div>
</content>
</entry>
<entry>
<title>rubocop -a</title>
<updated>2018-11-07T15:27:02+00:00</updated>
<author>
<name>SHIBATA Hiroshi</name>
<email>hsbt@ruby-lang.org</email>
</author>
<published>2018-11-07T15:27:02+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/bundler.git/commit/?id=79307d4aba2beb04045d3c84722f0f0498339d25'/>
<id>79307d4aba2beb04045d3c84722f0f0498339d25</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>To use around hook instead of before suite.</title>
<updated>2018-11-06T20:41:49+00:00</updated>
<author>
<name>SHIBATA Hiroshi</name>
<email>hsbt@ruby-lang.org</email>
</author>
<published>2018-11-06T20:41:49+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/bundler.git/commit/?id=35031a16b90e8883582319f78d396d7ba215c43d'/>
<id>35031a16b90e8883582319f78d396d7ba215c43d</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>To use helper method instead of env variables.</title>
<updated>2018-11-06T20:31:05+00:00</updated>
<author>
<name>SHIBATA Hiroshi</name>
<email>hsbt@ruby-lang.org</email>
</author>
<published>2018-11-06T20:31:05+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/bundler.git/commit/?id=8b1245ca967df4107d0f75ea20316d86fb63bed1'/>
<id>8b1245ca967df4107d0f75ea20316d86fb63bed1</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge #6714</title>
<updated>2018-11-06T09:23:46+00:00</updated>
<author>
<name>Bundlerbot</name>
<email>bot@bundler.io</email>
</author>
<published>2018-11-06T09:23:46+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/bundler.git/commit/?id=98040394795a8e5fd03d7e893f1a060041cd6777'/>
<id>98040394795a8e5fd03d7e893f1a060041cd6777</id>
<content type='text'>
6714: Spec reset state and refactorings r=deivid-rodriguez a=deivid-rodriguez

### What was the end-user problem that led to this PR?

The problem was that while working on #6713, I noticed several issues like order dependent failures, situations hard to debug, or specs doing too much and taking longer than they should.
 
### What was your diagnosis of the problem?

My diagnosis was that:

* Sometimes specs leak settings and env variable modifications.
* When a hang happens inside a subprocess, it's hard to debug because logging is not printed out anywhere.
* Some specs create unnecessary gemfiles and run `bundle install` multiple times unnecessarily.

### What is your fix for the problem, implemented in this PR?

My fix for state leaks is to reset state after each spec, for the hangs is to reorder some lines in the `sys_exec` helper (https://github.com/bundler/bundler/commit/b305a5b2524a6457b05a9d39e9526f75c98a0752), and for the unnecessary operations, to refactor the specs to avoid them.

### Why did you choose this fix out of the possible options?

I chose this fix because it seems like the best way to alleviate the issues found.


Co-authored-by: David Rodríguez &lt;deivid.rodriguez@riseup.net&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
6714: Spec reset state and refactorings r=deivid-rodriguez a=deivid-rodriguez

### What was the end-user problem that led to this PR?

The problem was that while working on #6713, I noticed several issues like order dependent failures, situations hard to debug, or specs doing too much and taking longer than they should.
 
### What was your diagnosis of the problem?

My diagnosis was that:

* Sometimes specs leak settings and env variable modifications.
* When a hang happens inside a subprocess, it's hard to debug because logging is not printed out anywhere.
* Some specs create unnecessary gemfiles and run `bundle install` multiple times unnecessarily.

### What is your fix for the problem, implemented in this PR?

My fix for state leaks is to reset state after each spec, for the hangs is to reorder some lines in the `sys_exec` helper (https://github.com/bundler/bundler/commit/b305a5b2524a6457b05a9d39e9526f75c98a0752), and for the unnecessary operations, to refactor the specs to avoid them.

### Why did you choose this fix out of the possible options?

I chose this fix because it seems like the best way to alleviate the issues found.


Co-authored-by: David Rodríguez &lt;deivid.rodriguez@riseup.net&gt;
</pre>
</div>
</content>
</entry>
</feed>
