Home
DevCentral
Search
Configure Global Search
Log In
Transactions
Code review at DevCentral
Change Details
Change Details
Old
New
Diff
**What to review?** The goal of a review is to craft a beautiful commit. - Code style - Documentation and comments proofread - Logic and edge cases - Tests - Commit message **Comment or amend?** Don't hesitate during a review to improve the commit. For that: - Commandeer revision on the web UI - `arc patch D...` - edit files - `arc diff` or `arc diff --update D...` **Self review is frowned upon** - If you're two or more in a project, every change should be reviewed - If this is a new repository or project, let the first commit be reviewed by the community to see if there is interest -
The goal of a review is to craft a beautiful commit. Everyone is welcome to participate, regardless experience. You can check code style, proofread documentation and comments, check logic and edge cases, add tests or ensure the commit message is crystal clear.
**What to review?**
The goal of a review is to craft a beautiful commit.
- Code style - Documentation and comments proofread - Logic and edge cases - Tests - Commit message **Comment or amend?** Don't hesitate during a review to improve the commit. For that: - Commandeer revision on the web UI - `arc patch D...` - edit files - `arc diff` or `arc diff --update D...` **Self review is frowned upon** - If you're two or more in a project
Everyone is welcome to participate
,
every change should be reviewed - If this is a new repository or project
regardless experience. You can check code style
,
let the first
proofread documentation and
comm
it be reviewed by the community to see if there is interest -
ents, check logic and edge cases, add tests or ensure the commit message is crystal clear.
Continue