Skip to content

🌈 Replace set usages with arrays#746

Merged
rafaelfranca merged 1 commit into
rails:masterfrom
deivid-rodriguez:replace_set_usage_with_arrays
Jan 20, 2021
Merged

🌈 Replace set usages with arrays#746
rafaelfranca merged 1 commit into
rails:masterfrom
deivid-rodriguez:replace_set_usage_with_arrays

Conversation

@deivid-rodriguez

Copy link
Copy Markdown
Contributor

Set has been gemified as of ruby 3.0.0. Since bundler vendors thor internally, it's helpful for bundler that thor doesn't use set, so that bundler doesn't activate the gem set too early and final users can still specify whichever version of set they want in their Gemfiles.

Set has been gemified as of ruby 3.0.0. Since `bundler` vendors `thor`
internally, it's helpful for `bundler` that `thor` doesn't use `set`, so
that `bundler` doesn't activate the gem `set` too early and final users
can still specify whichever version of `set` they want in their Gemfiles.
@deivid-rodriguez deivid-rodriguez changed the title Replace set usages with arrays 🌈 Replace set usages with arrays Jan 18, 2021
@deivid-rodriguez

Copy link
Copy Markdown
Contributor Author

I took the chance to make sure that bundler works with all recent changes in thor's master branch. All good 👍.

@rafaelfranca rafaelfranca merged commit e94bfe9 into rails:master Jan 20, 2021
@deivid-rodriguez

Copy link
Copy Markdown
Contributor Author

Thank you ❤️

@deivid-rodriguez deivid-rodriguez deleted the replace_set_usage_with_arrays branch January 20, 2021 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants