| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
6832: Bring CI matrix back to its previous state r=colby-swandale a=deivid-rodriguez
### What was the end-user problem that led to this PR?
The problem was that we lost some CI matrix entries after #6830.
### What was your diagnosis of the problem?
My diagnosis was that extra entries should be defined under `jobs:` now, specifying their stage.
### What is your fix for the problem, implemented in this PR?
My fix is to merge the `jobs` and `matrix` section, and specify the `stage` of each entry.
### Why did you choose this fix out of the possible options?
I chose this fix because it seems to bring back the matrix to the way it was before (except for the extra linting stage).
Co-authored-by: David Rodríguez <deivid.rodriguez@riseup.net>
|
|/ |
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| | |
6830: move rubocop into a separate stage in Travis-CI r=deivid-rodriguez a=colby-swandale
## Overview
This PR is pretty much a copy of https://github.com/rubygems/rubygems/pull/2510. It moves rubocop into a separate stage that is run before any tests are executed. This will allow for faster feedback and mean that we don't have to run rubocop on all our available instances in Travis.
Co-authored-by: Colby Swandale <me@colby.fyi>
|
|/ |
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
6818: Following up ruby core changes. r=hsbt a=hsbt
### What is your fix for the problem, implemented in this PR?
These changes enabled to use the same configuration of executables in ruby core repository.
* To use libexec instead of bin directory.
* Restore exe direcotry for bundler.gemspec while running test suite.
### Why did you choose this fix out of the possible options?
The ruby core repository uses `exe` directory for other usage. I chose `libexec` instead of `exe` in ruby core. And, The location of `bundler.gemspec` is not the root of the repository on ruby core. We should restore `libexec` to `exe` under the `lib` dir.
see. https://github.com/ruby/ruby/blob/trunk/lib/bundler.gemspec
Co-authored-by: SHIBATA Hiroshi <hsbt@ruby-lang.org>
|
| |
| |
| |
| |
| | |
* To use libexec instead of bin directory.
* Restore exe direcotry for bundler.gemspec while running test suite.
|
|\ \
| |/
|/|
| |
| |
| |
| |
| |
| |
| | |
6811: use '|' to preserve line breaks in welcome message r=colby-swandale a=colby-swandale
### What was the end-user problem that led to this PR?
I used the wrong block in the welcome message YAML. `>` strips the new line breaks, we want to use `|` instead.
Co-authored-by: Colby Swandale <me@colby.fyi>
|
|/ |
|
|\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
6798: Use https in the Github shortcut by default in Bundler 2 r=colby-swandale a=colby-swandale
This PR is setting Git gems to be fetched over https by default.
Note: This PR is successing #6791
Closes #6785
Co-authored-by: Colby Swandale <me@colby.fyi>
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
6805: [Plugin] Ignore the deployment setting r=colby-swandale a=segiddins
### What was the end-user problem that led to this PR?
The problem was installing a new plugin in deployment mode would fail with a suggestion the lock file was corrupted, when it installed fine without deployment mode.
Fixes #6795.
### What was your diagnosis of the problem?
My diagnosis was deployment mode was interfering with installing a new plugin.
### What is your fix for the problem, implemented in this PR?
My fix overrides the deployment and frozen settings when creating the plugin Definition.
### Why did you choose this fix out of the possible options?
I chose this fix because having a "frozen" plugin bundle makes no sense, since plugins aren't written to the lockfile.
Co-authored-by: Samuel Giddins <segiddins@segiddins.me>
|
| | |
| | |
| | |
| | | |
This allows installing plugins when the bundler is frozen, since plugins are not locked anyways
|
|\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | |
| | | |
6806: Todo remove require for Bundler 2.0 r=colby-swandale a=gavrieltal
Co-authored-by: Gavriel Loria <gtloria@protonmail.com>
|
| |\ \
| | | |
| | | |
| | | | |
https://github.com/gavrieltal/bundler into todo-remove-require-for-2.0
|
| | | | |
|
| |/ / |
|
|\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | |
| | | |
6804: move welcome message to new welcome bot r=colby-swandale a=colby-swandale
This PR is moving the functionality of the welcome bot to https://github.com/apps/welcome and will deprecate https://github.com/bundler/bundlerbot.
Co-authored-by: Colby Swandale <me@colby.fyi>
|
|/ / |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
6796: removed string dup in lockfile_parser.rb for bundler 2.0 r=segiddins a=gavrieltal
a task recommended by a TODO comment for the upgrade to Bundler 2.0
Co-authored-by: Gavriel Loria <gtloria@protonmail.com>
|
|/ / |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
6774: [Definition] Dont pin path deps to newer versions r=segiddins a=segiddins
### What was the end-user problem that led to this PR?
The problem was setting `only_update_to_newer_versions` would break using path gems.
Closes #6750
### What was your diagnosis of the problem?
My diagnosis was path gems should _always_ get their version used, since there's 0 alternative versions from a path source.
### What is your fix for the problem, implemented in this PR?
My fix should circuits the list of additional requirements when the dependency's source is a path source.
### Why did you choose this fix out of the possible options?
I chose this fix because it solves the problem in the same place that has the feature flag conditional
Co-authored-by: Samuel Giddins <segiddins@segiddins.me>
Co-authored-by: Grey Baker <greysteil@gmail.com>
|
| | |
| | |
| | |
| | | |
BUNDLE_BUNDLE_ONLY_UPDATE_TO_NEWER_VERSIONS
|
| | | |
|
|\ \ \
| |_|/
|/| |
| | |
| | |
| | |
| | |
| | |
| | | |
6794: Fix typo from `ssl_certs.rb` r=hsbt a=Mateus-Resende
There was a typo mistake in a comment inside `spec/bundler/ssl_certs/certificate_manager_spec.rb#14`.
Co-authored-by: Mateus Resende <mateus.resende@runtastic.com>
|
|/ / |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
6790: Fix multiple source warning messages from `error` to `warn` r=greysteil a=jlw
### What was the end-user problem that led to this PR?
I've been working on a few repos for most of this year that use a privately-hosted gem with the same name (but completely different purpose) as a gem hosted at rubygems.org - whenever I start with a clean copy and `bundle install` for the first time, I am presented with a warning in red that the gem was found in multiple sources. The red text makes me think that something went wrong, even though bundler did exactly what I wanted it to do.
The user experience is wrong - a warning should not be red.
### What was your diagnosis of the problem?
When I looked at the source, I saw immediately that the warning was using `Bundler.ui.error` instead of `Bundler.ui.warn`.
This is a bug fix with a limited lifespan - after setting up a local copy of bundler I see that 2.0's handling of multiple sources A) is smarter and bypasses the error in most of the repos where I saw it with bundler 1.x; and B) fails early with a different error message in the remaining repos.
### What is your fix for the problem, implemented in this PR?
The fix is simply switching to `Bundler.ui.warn` in this case.
### Why did you choose this fix out of the possible options?
_I didn't see any other possible options._
Co-authored-by: Jeremy Weathers <jeremy@codekindly.com>
|
| | | |
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
6786: Followed up #6743 r=hsbt a=hsbt
### What was the end-user problem that led to this PR?
#6743 was failed with ruby core repository and ignored working example.
I fixed them.
Co-authored-by: SHIBATA Hiroshi <hsbt@ruby-lang.org>
|
|/ / /
| | |
| | |
| | |
| | | |
#6743 was failed with ruby core repository and ignored working example.
Fixed them.
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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 <segiddins@segiddins.me>
Co-authored-by: Olle Jonsson <olle.jonsson@gmail.com>
Co-authored-by: Colby Swandale <me@colby.fyi>
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | | |
To make it clear the list is uniq’d
Co-Authored-By: segiddins <segiddins@segiddins.me>
|
| | |/
| |/| |
|
|\ \ \
| |_|/
|/| |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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 <hsbt@ruby-lang.org>
|
| | |
| | |
| | |
| | | |
spec/runtime/setup_spec.rb:896
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
them after finished bundler rspec.
|