ajvargo / ruby-refactor

A minor mode for Emacs that provides some Ruby refactoring methods.
63 stars 19 forks source link

auto select based on current line and test #27

Closed HParker closed 8 years ago

HParker commented 8 years ago

part of #12

I did two things here:

puts "some are greater than 3" unless long_method.empty?

previously became:

unless puts "some are greater than 3"
  long_method.empty?
end

now becomes:

unless long_method.empty?
  puts "some are greater than 3"
end

Let me know what you think. Thanks!

mikenichols commented 8 years ago

Awesome work, thanks man. When I was testing this, I got the reversed conditions once, but foolishly shrugged it off. Also :+1: for adding a test.

HParker commented 8 years ago

anytime. Anything else you would like to see here?

mikenichols commented 8 years ago

That's up to @ajvargo. He's been good at responding relatively quickly.

HParker commented 8 years ago

Cool. No hurry, just curious. :)