Skip to content

Commit c350143

Browse files
Merge pull request #8948 from rubygems/deivid-rodriguez/fix-did-you-mean-suggestions
Fix "did you mean" suggestions for unknown commands (cherry picked from commit e0d470b)
1 parent 17ea644 commit c350143

File tree

3 files changed

+4
-11
lines changed

3 files changed

+4
-11
lines changed

.github/workflows/ruby-core.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ jobs:
2727
with:
2828
repository: ruby/ruby
2929
path: ruby/ruby
30-
ref: 2cce628721728409a26c2d4732f63419785c7fd8 # TODO: Point to v3_4_5 once released
30+
ref: v3_4_5
3131
persist-credentials: false
3232
- name: Install libraries
3333
run: |

lib/rubygems/exceptions.rb

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -23,15 +23,8 @@ def initialize(unknown_command)
2323
def self.attach_correctable
2424
return if defined?(@attached)
2525

26-
if defined?(DidYouMean::SPELL_CHECKERS) && defined?(DidYouMean::Correctable)
27-
if DidYouMean.respond_to?(:correct_error)
28-
DidYouMean.correct_error(Gem::UnknownCommandError, Gem::UnknownCommandSpellChecker)
29-
else
30-
DidYouMean::SPELL_CHECKERS["Gem::UnknownCommandError"] =
31-
Gem::UnknownCommandSpellChecker
32-
33-
prepend DidYouMean::Correctable
34-
end
26+
if defined?(DidYouMean) && DidYouMean.respond_to?(:correct_error)
27+
DidYouMean.correct_error(Gem::UnknownCommandError, Gem::UnknownCommandSpellChecker)
3528
end
3629

3730
@attached = true

test/rubygems/test_gem_command_manager.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ def test_find_command_unknown_suggestions
7878

7979
message = "Unknown command pish".dup
8080

81-
if defined?(DidYouMean::SPELL_CHECKERS) && defined?(DidYouMean::Correctable)
81+
if defined?(DidYouMean)
8282
message << "\nDid you mean? \"push\""
8383
end
8484

0 commit comments

Comments
 (0)