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

Restore plugin autoloading as deprecated opt-in #2177

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

Conversation

bf4
Copy link

@bf4 bf4 commented Feb 16, 2021

lib/pry/pry_class.rb Outdated Show resolved Hide resolved
@bf4 bf4 force-pushed the add_deprecated_opt_in_plugin_autoloading branch from d02afed to 04e508c Compare February 16, 2021 01:07
@bf4 bf4 force-pushed the add_deprecated_opt_in_plugin_autoloading branch from 04e508c to 3538f23 Compare February 16, 2021 18:44
# TODO: Remove enable-plugins when removing plugin auto-loading
# Bring in options defined by plugins
Pry::Slop.new do
on "enable-plugins" do
Copy link
Author

Choose a reason for hiding this comment

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

not sure best way to test this

marcparadise added a commit to chef/chef that referenced this pull request Jun 28, 2021
This permits `pry-byebug` to continue working with it, until
either pry/pry#2177 is merged,
or  deivid-rodriguez/pry-byebug#343 is
otherwise resolved.

Signed-off-by: Marc A. Paradise <marc.paradise@gmail.com>
marcparadise added a commit to chef/chef that referenced this pull request Jun 28, 2021
This permits `pry-byebug` to continue working with it, until
either pry/pry#2177 is merged,
or  deivid-rodriguez/pry-byebug#343 is
otherwise resolved.

Signed-off-by: Marc A. Paradise <marc.paradise@gmail.com>
johnmccrae pushed a commit to chef/chef that referenced this pull request Jul 16, 2021
This permits `pry-byebug` to continue working with it, until
either pry/pry#2177 is merged,
or  deivid-rodriguez/pry-byebug#343 is
otherwise resolved.

Signed-off-by: Marc A. Paradise <marc.paradise@gmail.com>
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.

None yet

1 participant