Skip to content

Refactoring by template - #170

Open
dukaev wants to merge 1 commit into
fastruby:mainfrom
dukaev:refactoring-position-slow-fast
Open

Refactoring by template#170
dukaev wants to merge 1 commit into
fastruby:mainfrom
dukaev:refactoring-position-slow-fast

Conversation

@dukaev

@dukaev dukaev commented Feb 13, 2019 •
edited
Loading

Copy link
Copy Markdown
@dukaev
Did refactoring by template:
4185266 
require "benchmark/ips"

def fast
end

def slow
end

Benchmark.ips do |x|
  x.report("fast code description") { fast }
  x.report("slow code description") { slow }
  x.compare!
end
@dukaev dukaev changed the title Did refactoring by template Feb 13, 2019
mateusdeap
mateusdeap reviewed Oct 12, 2022
View reviewed changes

@mateusdeap mateusdeap left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@dukaev I don't know if you're already on this, but I believe that if you just add Ashwini's suggestions to your PR, in the issue this solves, we can merge it.

Thanks for the good work!
@mateusdeap

mateusdeap commented Oct 12, 2022

Copy link
Copy Markdown
Member

If not, that's fine. Just let me know and I can take it over.
@dukaev

dukaev commented Oct 15, 2022

Copy link
Copy Markdown
Author

@mateusdeap great. In a few days I will do this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@dukaev @mateusdeap