diff --git a/checklists/code_review_checklist.md b/checklists/code_review_checklist.md index 66705bd3..9b8e4f5d 100644 --- a/checklists/code_review_checklist.md +++ b/checklists/code_review_checklist.md @@ -454,6 +454,10 @@ addListener: function( listener ) { /*...*/ } * Regex for property assignment like `x.y = something`: `[\w]+\.[\w]+\s=` * Regex for function declarations: `[\w]+: function\(` +#### **Math Libraries** + +- [ ] `DOT/Util.toFixed` or `DOT/Util.toFixedNumber` should be used instead of `toFixed`. JavaScript's `toFixed` is notoriously buggy. Behavior differs depending on browser, because the spec doesn't specify whether to round or floor. + #### IE11 - [ ] No usage of `Number.parseInt()` - [ ] No usage of `Array.prototype.find`