ruby-core@ruby-lang.org archive (unofficial mirror)
 help / color / mirror / Atom feed
* [ruby-core:95917] [Ruby master Bug#16363] Promote did_you_mean to default gem
       [not found] <redmine.issue-16363.20191124002744@ruby-lang.org>
@ 2019-11-24  0:27 ` mail
  2019-11-24  3:09 ` [ruby-core:95920] " shevegen
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 5+ messages in thread
From: mail @ 2019-11-24  0:27 UTC (permalink / raw
  To: ruby-core

Issue #16363 has been reported by yuki24 (Yuki Nishijima).

----------------------------------------
Bug #16363: Promote did_you_mean to default gem
https://bugs.ruby-lang.org/issues/16363

* Author: yuki24 (Yuki Nishijima)
* Status: Open
* Priority: Normal
* Assignee: 
* Target version: 
* ruby -v: 
* Backport: 2.5: UNKNOWN, 2.6: UNKNOWN
----------------------------------------
The `did_you_mean` gem has been a bundled gem for over 3 years now and I think it's time to promote it to a standard library. There are two main issues in the current structure:

 * A number of implementations, including [`optparse.rb`](https://github.com/ruby/ruby/blob/fb6a489af2765a3b56e301adf0019af6bbad6156/lib/optparse.rb#L1792), [rake](https://github.com/ruby/rake/blob/985abffa9954d21790831d9626d9c38b24a94199/lib/rake/task_manager.rb#L69-L78), and [rails](https://github.com/rails/rails/blob/98a57aa5f610bc66af31af409c72173cdeeb3c9e/railties/lib/rails/command/spellchecker.rb#L8), now take advantage of the gem's spell checker, but it's awkward to have to check the existence of the `DidYouMean::SpellChecker` constant.
 * there are some problems with regard to bundler + did_you_mean because of did_you_mean being a bundled gem. Since the vendored version of thor inside bundler and ruby itself explicitly requires did_you_mean, it can become difficult to load it when using Bundler.setup. See this issue: https://github.com/yuki24/did_you_mean/issues/117#issuecomment-482733159 for more details.

I would like to promote the gem up to a standard library and make it always available, so we can always reliably require it whenever we want to.

A patch could be found here: https://github.com/ruby/ruby/pull/2689



-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [ruby-core:95920] [Ruby master Bug#16363] Promote did_you_mean to default gem
       [not found] <redmine.issue-16363.20191124002744@ruby-lang.org>
  2019-11-24  0:27 ` [ruby-core:95917] [Ruby master Bug#16363] Promote did_you_mean to default gem mail
@ 2019-11-24  3:09 ` shevegen
  2019-11-28  7:01 ` [ruby-core:96014] " matz
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 5+ messages in thread
From: shevegen @ 2019-11-24  3:09 UTC (permalink / raw
  To: ruby-core

Issue #16363 has been updated by shevegen (Robert A. Heiler).


\o/

(^^^ That's an upvote emoji really.)

----------------------------------------
Bug #16363: Promote did_you_mean to default gem
https://bugs.ruby-lang.org/issues/16363#change-82760

* Author: yuki24 (Yuki Nishijima)
* Status: Open
* Priority: Normal
* Assignee: 
* Target version: 
* ruby -v: 
* Backport: 2.5: UNKNOWN, 2.6: UNKNOWN
----------------------------------------
The `did_you_mean` gem has been a bundled gem for over 3 years now and I think it's time to promote it to a standard library. There are two main issues in the current structure:

 * A number of implementations, including [`optparse.rb`](https://github.com/ruby/ruby/blob/fb6a489af2765a3b56e301adf0019af6bbad6156/lib/optparse.rb#L1792), [rake](https://github.com/ruby/rake/blob/985abffa9954d21790831d9626d9c38b24a94199/lib/rake/task_manager.rb#L69-L78), and [rails](https://github.com/rails/rails/blob/98a57aa5f610bc66af31af409c72173cdeeb3c9e/railties/lib/rails/command/spellchecker.rb#L8), now take advantage of the gem's spell checker, but it's awkward to have to check the existence of the `DidYouMean::SpellChecker` constant.
 * there are some problems with regard to bundler + did_you_mean because of did_you_mean being a bundled gem. Since the vendored version of thor inside bundler and ruby itself explicitly requires did_you_mean, it can become difficult to load it when using Bundler.setup. See this issue: https://github.com/yuki24/did_you_mean/issues/117#issuecomment-482733159 for more details.

I would like to promote the gem up to a standard library and make it always available, so we can always reliably require it whenever we want to.

A patch could be found here: https://github.com/ruby/ruby/pull/2689



-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [ruby-core:96014] [Ruby master Bug#16363] Promote did_you_mean to default gem
       [not found] <redmine.issue-16363.20191124002744@ruby-lang.org>
  2019-11-24  0:27 ` [ruby-core:95917] [Ruby master Bug#16363] Promote did_you_mean to default gem mail
  2019-11-24  3:09 ` [ruby-core:95920] " shevegen
@ 2019-11-28  7:01 ` matz
  2019-12-06 13:26 ` [ruby-core:96127] " v.ondruch
  2019-12-09  1:15 ` [ruby-core:96146] [Ruby master Feature#16363] " hsbt
  4 siblings, 0 replies; 5+ messages in thread
From: matz @ 2019-11-28  7:01 UTC (permalink / raw
  To: ruby-core

Issue #16363 has been updated by matz (Yukihiro Matsumoto).


Accepted. Go ahead.

Matz.


----------------------------------------
Bug #16363: Promote did_you_mean to default gem
https://bugs.ruby-lang.org/issues/16363#change-82854

* Author: yuki24 (Yuki Nishijima)
* Status: Open
* Priority: Normal
* Assignee: 
* Target version: 
* ruby -v: 
* Backport: 2.5: UNKNOWN, 2.6: UNKNOWN
----------------------------------------
The `did_you_mean` gem has been a bundled gem for over 3 years now and I think it's time to promote it to a standard library. There are two main issues in the current structure:

 * A number of implementations, including [`optparse.rb`](https://github.com/ruby/ruby/blob/fb6a489af2765a3b56e301adf0019af6bbad6156/lib/optparse.rb#L1792), [rake](https://github.com/ruby/rake/blob/985abffa9954d21790831d9626d9c38b24a94199/lib/rake/task_manager.rb#L69-L78), and [rails](https://github.com/rails/rails/blob/98a57aa5f610bc66af31af409c72173cdeeb3c9e/railties/lib/rails/command/spellchecker.rb#L8), now take advantage of the gem's spell checker, but it's awkward to have to check the existence of the `DidYouMean::SpellChecker` constant.
 * there are some problems with regard to bundler + did_you_mean because of did_you_mean being a bundled gem. Since the vendored version of thor inside bundler and ruby itself explicitly requires did_you_mean, it can become difficult to load it when using Bundler.setup. See this issue: https://github.com/yuki24/did_you_mean/issues/117#issuecomment-482733159 for more details.

I would like to promote the gem up to a standard library and make it always available, so we can always reliably require it whenever we want to.

A patch could be found here: https://github.com/ruby/ruby/pull/2689



-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [ruby-core:96127] [Ruby master Bug#16363] Promote did_you_mean to default gem
       [not found] <redmine.issue-16363.20191124002744@ruby-lang.org>
                   ` (2 preceding siblings ...)
  2019-11-28  7:01 ` [ruby-core:96014] " matz
@ 2019-12-06 13:26 ` v.ondruch
  2019-12-09  1:15 ` [ruby-core:96146] [Ruby master Feature#16363] " hsbt
  4 siblings, 0 replies; 5+ messages in thread
From: v.ondruch @ 2019-12-06 13:26 UTC (permalink / raw
  To: ruby-core

Issue #16363 has been updated by vo.x (Vit Ondruch).


I think we should go rather in opposite direction. Move all gems to "bundled" gems. Especially "did_you_mean" gem does not provide any benefit to **end user** of application and it should not be used in runtime. It is just perfectly fine when its functionality is optional.

BTW since the PR was merged, this ticket should have been closed.

----------------------------------------
Bug #16363: Promote did_you_mean to default gem
https://bugs.ruby-lang.org/issues/16363#change-82993

* Author: yuki24 (Yuki Nishijima)
* Status: Open
* Priority: Normal
* Assignee: 
* Target version: 
* ruby -v: 
* Backport: 2.5: UNKNOWN, 2.6: UNKNOWN
----------------------------------------
The `did_you_mean` gem has been a bundled gem for over 3 years now and I think it's time to promote it to a standard library. There are two main issues in the current structure:

 * A number of implementations, including [`optparse.rb`](https://github.com/ruby/ruby/blob/fb6a489af2765a3b56e301adf0019af6bbad6156/lib/optparse.rb#L1792), [rake](https://github.com/ruby/rake/blob/985abffa9954d21790831d9626d9c38b24a94199/lib/rake/task_manager.rb#L69-L78), and [rails](https://github.com/rails/rails/blob/98a57aa5f610bc66af31af409c72173cdeeb3c9e/railties/lib/rails/command/spellchecker.rb#L8), now take advantage of the gem's spell checker, but it's awkward to have to check the existence of the `DidYouMean::SpellChecker` constant.
 * there are some problems with regard to bundler + did_you_mean because of did_you_mean being a bundled gem. Since the vendored version of thor inside bundler and ruby itself explicitly requires did_you_mean, it can become difficult to load it when using Bundler.setup. See this issue: https://github.com/yuki24/did_you_mean/issues/117#issuecomment-482733159 for more details.

I would like to promote the gem up to a standard library and make it always available, so we can always reliably require it whenever we want to.

A patch could be found here: https://github.com/ruby/ruby/pull/2689



-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [ruby-core:96146] [Ruby master Feature#16363] Promote did_you_mean to default gem
       [not found] <redmine.issue-16363.20191124002744@ruby-lang.org>
                   ` (3 preceding siblings ...)
  2019-12-06 13:26 ` [ruby-core:96127] " v.ondruch
@ 2019-12-09  1:15 ` hsbt
  4 siblings, 0 replies; 5+ messages in thread
From: hsbt @ 2019-12-09  1:15 UTC (permalink / raw
  To: ruby-core

Issue #16363 has been updated by hsbt (Hiroshi SHIBATA).

Backport deleted (2.5: UNKNOWN, 2.6: UNKNOWN)
Assignee set to yuki24 (Yuki Nishijima)
Tracker changed from Bug to Feature

----------------------------------------
Feature #16363: Promote did_you_mean to default gem
https://bugs.ruby-lang.org/issues/16363#change-83015

* Author: yuki24 (Yuki Nishijima)
* Status: Closed
* Priority: Normal
* Assignee: yuki24 (Yuki Nishijima)
* Target version: 
----------------------------------------
The `did_you_mean` gem has been a bundled gem for over 3 years now and I think it's time to promote it to a standard library. There are two main issues in the current structure:

 * A number of implementations, including [`optparse.rb`](https://github.com/ruby/ruby/blob/fb6a489af2765a3b56e301adf0019af6bbad6156/lib/optparse.rb#L1792), [rake](https://github.com/ruby/rake/blob/985abffa9954d21790831d9626d9c38b24a94199/lib/rake/task_manager.rb#L69-L78), and [rails](https://github.com/rails/rails/blob/98a57aa5f610bc66af31af409c72173cdeeb3c9e/railties/lib/rails/command/spellchecker.rb#L8), now take advantage of the gem's spell checker, but it's awkward to have to check the existence of the `DidYouMean::SpellChecker` constant.
 * there are some problems with regard to bundler + did_you_mean because of did_you_mean being a bundled gem. Since the vendored version of thor inside bundler and ruby itself explicitly requires did_you_mean, it can become difficult to load it when using Bundler.setup. See this issue: https://github.com/yuki24/did_you_mean/issues/117#issuecomment-482733159 for more details.

I would like to promote the gem up to a standard library and make it always available, so we can always reliably require it whenever we want to.

A patch could be found here: https://github.com/ruby/ruby/pull/2689



-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2019-12-09  1:15 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <redmine.issue-16363.20191124002744@ruby-lang.org>
2019-11-24  0:27 ` [ruby-core:95917] [Ruby master Bug#16363] Promote did_you_mean to default gem mail
2019-11-24  3:09 ` [ruby-core:95920] " shevegen
2019-11-28  7:01 ` [ruby-core:96014] " matz
2019-12-06 13:26 ` [ruby-core:96127] " v.ondruch
2019-12-09  1:15 ` [ruby-core:96146] [Ruby master Feature#16363] " hsbt

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).