Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve recent force_ruby_platform fix #7115

Merged
merged 2 commits into from Oct 31, 2023

Conversation

deivid-rodriguez
Copy link
Member

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

The developer problem is small:

  • Potential to make code a bit simpler.
  • Potential to make both spec wording and implementation easier to understand.

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

This PR iterates on #6936.

Feedback in the PR was not blocking, but I think it has merit to be introduced.

Make sure the following tasks are checked

bundler/lib/bundler/gem_helpers.rb Outdated Show resolved Hide resolved
@deivid-rodriguez deivid-rodriguez force-pushed the improve-one-force-ruby-platform-spec branch from 5c6bc07 to 3e7f137 Compare October 27, 2023 21:50
Copy link
Member

@martinemde martinemde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Neat, I forgot all about that older comment that I made on this code. Glad it could help and be revived. As long as the modified test still covers the case we're trying to fix, then this looks good.

@deivid-rodriguez deivid-rodriguez merged commit 03213db into master Oct 31, 2023
92 checks passed
@deivid-rodriguez deivid-rodriguez deleted the improve-one-force-ruby-platform-spec branch October 31, 2023 10:48
deivid-rodriguez added a commit that referenced this pull request Nov 8, 2023
…m-spec

Improve recent `force_ruby_platform` fix

(cherry picked from commit 03213db)
deivid-rodriguez added a commit that referenced this pull request Nov 8, 2023
…m-spec

Improve recent `force_ruby_platform` fix

(cherry picked from commit 03213db)
deivid-rodriguez added a commit that referenced this pull request Nov 8, 2023
…m-spec

Improve recent `force_ruby_platform` fix

(cherry picked from commit 03213db)
deivid-rodriguez added a commit that referenced this pull request Nov 8, 2023
…m-spec

Improve recent `force_ruby_platform` fix

(cherry picked from commit 03213db)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants