Skip to content

Conversation

deivid-rodriguez
Copy link
Contributor

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

In #8913, I finally understood an old plan to remove bundle show. bundle show is a bit confusing and we have more specific commands to do what it does. When passed a specific gem bundle show <gem-name>, we have bundle info as a better alternative. When not passed a specific gem, we have bundle list as a better alternative.

We could eventually deprecate bundle show, but we should make sure all functionality has some replacement. However, bundle show --outdated does not have a bundle list equivalent.

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

This PR copies bundle show --outdated implementation to bundle list --verbose and recommends that as an alternative in the deprecation message.

Make sure the following tasks are checked

@deivid-rodriguez deivid-rodriguez force-pushed the deivid-rodriguez/improve-bundle-show-outdated-deprecation branch from 7042a7d to 5194659 Compare August 8, 2025 12:02
@deivid-rodriguez deivid-rodriguez changed the title Add a bundle list --verbose equivalent to bundle show --verbose and bundle show --outdated, and recommend it as a bundle show --outdated replacement Fix bundle show --verbose and recommend it as an alternative to bundle show --outdated Aug 8, 2025
@deivid-rodriguez
Copy link
Contributor Author

deivid-rodriguez commented Aug 8, 2025

Ok, just more little change. I realized bundle show --verbose had a bug, so I fixed it and I'm recommending that in favor of bundle show --outdated.

We can later figure out a plan to deprecate bundle show but for now there should be no need to add features to bundle list (which, by the way, would conflict with #8728). I'm now happy with this PR 👍.

@deivid-rodriguez deivid-rodriguez force-pushed the deivid-rodriguez/improve-bundle-show-outdated-deprecation branch from 5194659 to 98d35d6 Compare August 8, 2025 12:17
We had actually cancelled this change. After a second look though, it
does seem like a reasonable plan since `bundle list` makes more sense
for listing gems, and `bundle info` makes more sense for showing info
about a particular gem. `bundle show` is a strange mix of both.

Let's schedule this for Bundler 5.
@deivid-rodriguez deivid-rodriguez force-pushed the deivid-rodriguez/improve-bundle-show-outdated-deprecation branch from 98d35d6 to 7071a1e Compare August 11, 2025 09:29
@deivid-rodriguez
Copy link
Contributor Author

There was a pending spec that was assuming that we'd remove bundle show in Bundler 4. We had canceled that change, but removal makes sense to me after all, so I delayed that pending spec to Bundler 5 instead of removing it.

@deivid-rodriguez deivid-rodriguez merged commit a49d940 into master Aug 11, 2025
76 checks passed
@deivid-rodriguez deivid-rodriguez deleted the deivid-rodriguez/improve-bundle-show-outdated-deprecation branch August 11, 2025 10:57
deivid-rodriguez added a commit that referenced this pull request Sep 4, 2025
…e-show-outdated-deprecation

Fix `bundle show --verbose` and recommend it as an alternative to `bundle show --outdated`

(cherry picked from commit a49d940)
deivid-rodriguez added a commit that referenced this pull request Sep 9, 2025
…e-show-outdated-deprecation

Fix `bundle show --verbose` and recommend it as an alternative to `bundle show --outdated`

(cherry picked from commit a49d940)
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.

1 participant