Skip to content
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

Refactored Default Key Bindings #678

Merged
merged 3 commits into from Apr 14, 2024
Merged

Conversation

ima1zumi
Copy link
Member

I checked GNU Readline documentation and defined key bindings for existing features to match Readline.

Also, I removed key bindings that have key bindings defined but no methods defined.

Removed key bindings:

  • em_copy_prev_word
  • ed_sequence_lead_in
  • em_copy_region
  • ed_command
  • vi_match
  • vi_repeat_prev_char
  • vi_redo
  • vi_repeat_search_prev
  • vi_replace_mode
  • vi_substitute_line
  • vi_undo_line
  • vi_yank_end
  • vi_history_word
  • vi_repeat_search_next
  • vi_substitute_char
  • vi_undo
  • vi_change_case

Copy link
Member

@tompng tompng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@tompng tompng merged commit 155f704 into ruby:master Apr 14, 2024
40 checks passed
matzbot pushed a commit to ruby/ruby that referenced this pull request Apr 14, 2024
(ruby/reline#678)

* Reduce duplicate method

* Configured default key mapping with Readline when the method exists

* Remove undefined methods

ruby/reline@155f7047bb
@ima1zumi ima1zumi deleted the set-default-key-binding branch April 15, 2024 13:21
tenderlove added a commit to tenderlove/ruby that referenced this pull request Apr 15, 2024
* master: (29 commits)
  Not all `nm`s support the `--help` option
  Emit a performance warning when redefining specially optimized methods
  YJIT: A64: Avoid intermediate register in `opt_and` and friends (ruby#10509)
  [ruby/reline] Remove not implemented method (ruby/reline#680)
  [ruby/reline] Fix vi_to_column which was broken (ruby/reline#679)
  Include more debug information in test_uplus_minus
  [Universal parser] DeVALUE of p->debug_lines and ast->body.script_lines
  Add more assertions in `test_uplus_minus`
  `super{}` doesn't use block
  fix incorrect warning.
  Update default gems list at fc36394 [ci skip]
  [ruby/optparse] bump up to 0.5.0
  show warning for unused block
  Emit `warn` event for duplicated hash keys on ripper
  [ruby/reline] Refactored Default Key Bindings (ruby/reline#678)
  [ruby/reline] Refactor waiting_proc and waiting_operator_proc (ruby/reline#649)
  [pty] Fix missing `or`
  [pty] Fix `ptsname_r` fallback
  [ruby/irb] Allow defining custom commands in IRB (ruby/irb#886)
  [ruby/reline] Support `menu-complete-backward` command for upward navigation (ruby/reline#677)
  ...
artur-intech pushed a commit to artur-intech/ruby that referenced this pull request Apr 26, 2024
(ruby/reline#678)

* Reduce duplicate method

* Configured default key mapping with Readline when the method exists

* Remove undefined methods

ruby/reline@155f7047bb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants