Github user garrensmith commented on the pull request:
https://github.com/apache/couchdb-fauxton/pull/91#issuecomment-58860976
This is great and needed. Two things, we talk a fair bit of for loops and
the correct way of implementing them. I think we use a for loop in a handful of
places. I think we should mention that we prefer using one of underscore's
array/collection methods instead of for loops. In most cases using a for loop
is a code smell.
We should also mention that we try to avoid using `var that = this` and
instead try and use `_.bind` to set the context for the method being called.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
