For a good PR review you should consider the following topics
- It's proposal should be clear
- Code quality
- Readability
- Unexpected exception
- Security Issues
- Unexpected recursion
- Test readability
- Test coverage
For a good PR review you should consider the following topics
FROM alpine:3.2 | |
MAINTAINER RakutenPayments <[email protected]> | |
RUN \ | |
echo 'gem: --no-document' >> ~/.gemrc && \ | |
cp ~/.gemrc /etc/gemrc && \ | |
chmod uog+r /etc/gemrc | |
ENV \ |
Get Git log in JSON format
git log --pretty=format:'{%n "commit": "%H",%n "abbreviated_commit": "%h",%n "tree": "%T",%n "abbreviated_tree": "%t",%n "parent": "%P",%n "abbreviated_parent": "%p",%n "refs": "%D",%n "encoding": "%e",%n "subject": "%s",%n "sanitized_subject_line": "%f",%n "body": "%b",%n "commit_notes": "%N",%n "verification_flag": "%G?",%n "signer": "%GS",%n "signer_key": "%GK",%n "author": {%n "name": "%aN",%n "email": "%aE",%n "date": "%aD"%n },%n "commiter": {%n "name": "%cN",%n "email": "%cE",%n "date": "%cD"%n }%n},'
The only information that aren't fetched are:
%B
: raw body (unwrapped subject and body)%GG
: raw verification message from GPG for a signed commitrequire 'capybara/rails' | |
Capybara.app = Rack::Builder.new do | |
map "some_nested_url" do | |
run Rails.application | |
end | |
end.to_app |
#!/usr/bin/env ruby | |
# An example hook script to verify what is about to be pushed. Called by "git | |
# push" after it has checked the remote status, but before anything has been | |
# pushed. If this script exits with a non-zero status nothing will be pushed. | |
# This script shows a disclaimer with unsetted env vars on your deis cluster, | |
# only when you do a "git push deis master" | |
# | |
# It tries to read a ".env.sample" file to diff with your current set of env |
Consideração pelo seu trabalho
pergunte! questione! Seja crítico.
PENSE! Nunca ande no piloto automático! Critique seu trabalho em tempo real
Assuma a responsabilidade! Seja realista!
Dê soluções, não desculpas.
quando não se conhece a si mesmo nem o seu inimigo, suas chances de ganhar as batalhas são próximas de zero |
http://linuxlegal.blogspot.com.br/2014/02/cedilha-no-ubuntu-1310-com-teclado.html | |
# on debian just edit the `/etc/default/keyboard` with: | |
XKBMODEL="pc105" | |
XKBLAYOUT="us" | |
XKBVARIANT="intl" | |
XKBOPTIONS="" |
from: https://github.com/thoughtbot/til/pull/86/files
I recently had to replace static files path in multiple files from a project I were working on, after boggling my mind trying to do it with sed, I gave up and looked up how to do it using VIM, I was sure there was a way.
When VIM is started, you can specify multiple files to open as buffers from the