Skip to content

Commit

Permalink
Fix undefined method `windows?'
Browse files Browse the repository at this point in the history
This fixes an incompatibility with current pry HEAD, which apparently retired Pry::Platform. 

I fear this fix by itself would again be incompatible with current pry *release* version. But I don't know enough about the respective releases processes, and which strategy you tend to prefer (set dependency to next pry release/```respond_to?```/etc.).

Feel free to close and/or adapt as you see fit.
  • Loading branch information
MatthiasWinkelmann authored Nov 1, 2018
1 parent 1e77c7c commit 679b134
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/pry-rescue.rb
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
end
case ENV['PRY_PEEK']
when nil
PryRescue.peek_on_signal('QUIT') unless Pry::Helpers::BaseHelpers.windows?
PryRescue.peek_on_signal('QUIT') unless Pry::Helpers::Platform.windows?
when ''
# explicitly disable QUIT.
else
Expand Down

0 comments on commit 679b134

Please sign in to comment.