-
-
Notifications
You must be signed in to change notification settings - Fork 24
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
Update unicode-display_width #60
Comments
PR here: #61 |
Thank you for submitting this issue.
I believe you meant
When I replicate a minimal scenario with
I have nothing against relaxing the upper constraint to include any |
Yes, my mistake, I did mean rubocop.
It happens when rubocop is installed first. i.e. if you add tty-progressbar to an existing project, it will install the old version.
Note that tty-progressbar 0.14.0 is installed.
I created a PR here: #61 - Looks like older ruby versions are no longer supported. Given that the minimum supported version of Ruby 2.4 was released 8+ years ago, perhaps it would be good to drop the CI checks for older rubies? |
Fair enough, the bundler resolution prioritises the existing dependencies.
Great!
All the tty gems support Ruby 2.0 or higher. That's a deliberate choice. It embraces the nature of terminal applications and their distribution. Many gems are distributed and built by various Linux distros. Not using the latest features keeps internals more portable. When I started writing tty gems more than a decade ago, it was to make CLI apps easy to build. But also as I was creating the gems, I wanted to have an easy way to build self-contained executables similar to Go. Part of that idea was being able to port the gems to mruby. This would allow the gems to be used, for example, within embedded systems. Therefore, I won't be dropping support for older versions for now. |
The current gem dependencies restricts unicode-display_width to ">= 1.6", "< 3.0".
In my project, we use rspec, which allows unicode-display_width >= 2.4.0, < 4.0.
This means our project uses the latest unicode-display_width, 3.1.4.
So when I try to install tty-progressbar, it installs version 0.14.0, instead of the latest.
Can tty-progressbar be updated to work with unicode-display_width <4.0 ?
The text was updated successfully, but these errors were encountered: