Skip to content

Make the progressbar and ruby-progressbar gems only differ in their names, but have the exact same content. #200

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

npf
Copy link

@npf npf commented May 4, 2025


name: Pull Request
about: Make the progressbar and ruby-progressbar gems only differ in their names, but have the exact same content.


Let both the ruby-progressbar and progressbar gems allow requiring progressbar or ruby-progessbar indifferently.

Why: Having the 2 gems with quite the same content but not exactly is very
confusing, for instance, on Debian because the ruby-progressbar deb
package contains the progressbar gem, not the ruby-progressbar
gem.

IMHO, having both gems with the exact same content would be more
user-friendly, allowing 'require "progressbar"' and 'require
"ruby-progressbar"' indifferently, independently of what gem is actually installed
(be it with gem install or a distribution package).

npf added 2 commits May 4, 2025 13:01
….rb or ruby-progessbar.rb

Having the 2 gems with quite the same content but not exactly is very
confusing, for instance on Debian because the ruby-progressbar deb
package actually contains the progressbar gem, not the ruby-progressbar
gem.

IMHO, having both gems with the exact same content would be more
user-friendly, allowing 'require "progressbar"' and 'require
"ruby-progressbar"' indifferently from what gem is actually installed
(be it with gem install or a distrubution package).
…ntent)

It's the same content.
Removes the duplication of code.
Avoid possible divergence.
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.

1 participant