Adds documentation entry: Don't user forEach, aim for code without side effects
What does this MR do?
Adds documentation entry about forEach and side effects.
forEach will cause side effects, we are aiming for no side effects code.
As suggested by @timzallmann it would be good to have documentation with Do's and Don'ts that we can refer to.