Skip to content
This repository has been archived by the owner on Apr 14, 2021. It is now read-only.

Track changes from ruby core master #7301

Merged
9 commits merged into from Aug 16, 2019
Merged

Track changes from ruby core master #7301

9 commits merged into from Aug 16, 2019

Conversation

hsbt
Copy link
Member

@hsbt hsbt commented Aug 14, 2019

What was the end-user problem that led to this PR?

I'm going to merge #7274. But the ruby-core source has some of the changes for bundler source.

What was your diagnosis of the problem?

What is your fix for the problem, implemented in this PR?

ruby core team fixed them:

  • Removed circular require warning at shared_helper.rb
  • Support test at GitHub Actions, It helps that bundler will migrate Actions from Azure Pipelines too.
  • Fixed broken examples at ruby core repository

Why did you choose this fix out of the possible options?

@deivid-rodriguez
Copy link
Member

Thank you @hsbt, go ahead and merge when CI is green 👍.

@deivid-rodriguez
Copy link
Member

Rubocop is failing because the double negation style (!!) is not used in the bundler repo.

@hsbt
Copy link
Member Author

hsbt commented Aug 15, 2019

I will fix them later.

@hsbt
Copy link
Member Author

hsbt commented Aug 16, 2019

@bundlerbot r+

ghost pushed a commit that referenced this pull request Aug 16, 2019
7301: Track changes from ruby core master r=hsbt a=hsbt

### What was the end-user problem that led to this PR?

I'm going to merge #7274. But the ruby-core source has some of the changes for bundler source.

### What was your diagnosis of the problem?


### What is your fix for the problem, implemented in this PR?

ruby core team fixed them:

* Removed circular require warning at `shared_helper.rb`
* Support test at GitHub Actions, It helps that bundler will migrate Actions from Azure Pipelines too.
* Fixed broken examples at ruby core repository


### Why did you choose this fix out of the possible options?



Co-authored-by: ohbarye <over.rye@gmail.com>
Co-authored-by: Hiroshi SHIBATA <hsbt@ruby-lang.org>
Co-authored-by: Yusuke Endoh <mame@ruby-lang.org>
@ghost
Copy link

ghost commented Aug 16, 2019

Build succeeded

@ghost ghost merged commit 9256177 into master Aug 16, 2019
@ghost ghost deleted the track-ruby-core-master branch August 16, 2019 00:42
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants