Confirmed users
709
edits
Spreadfirefox: Difference between revisions
→Do's and Don'ts
| Line 38: | Line 38: | ||
** Function definition blocks at the least | ** Function definition blocks at the least | ||
** Inline comments for complex code, use your best judgement | ** Inline comments for complex code, use your best judgement | ||
* Always put a bug # and comment in your commit messages. | |||
* Don't embed HTML code in module files. | |||
** Define templates and keep presentation separate from logic code. | |||
* Don't touch the Drupal core. If you think you absolutely have to, ok it with another developer and document where and why you did so. | * Don't touch the Drupal core. If you think you absolutely have to, ok it with another developer and document where and why you did so. | ||
* Don't push on Fridays or when you won't be around to verify the changes. | * Don't push on Fridays or when you won't be around to verify the changes. | ||