class Human
end
class Person < Human
end
person = Person.new
-
This gist contains a PR description template.
-
Copy its content to your PRs and replace all TODOs with useful information (Scroll to the bottom to see its preview).
-
Depending on the complexity of the PR some sections may be completely skipped. Make a decision based on your experience.
require "byebug" if (defined? ::Rails::Console)
(require "#{`bundle info byebug --path`.chomp}/lib/byebug.rb"; byebug) if (defined? ::Rails::Console)
(require `gem which byebug`.chomp; byebug) if (defined? ::Rails::Console)
Status | 🚧 |
Tags | code-review , principles |
-
Always try to get familiar with the problem domain at least for some adequate time period before asking questions (make an initial investigation, check existing codebase, find confluence pages, read the docs, prepare a list of concrete questions, etc).
-
Try to predict possible edge cases by yourself (you don't need to implement all of them, but you should be aware of the probability).
Just download the HTML page you like and open it in your browser.
Here is an example using cURL (please, scroll horizontally to see the full command, I could place rm -rf *
in the end 🤯):
curl https://gist.githubusercontent.com/marian13/504631acfbfbeecc2503c0cd7614ca9c/raw/d0368d93090aea88323a184e7ef43b10db24f667/convenient_service_diagrams_method_middlewares_sequence_diagram.html -o diagram.html && open diagram.html