The diff generation here goofed me up and at first I thought this was changing the actual logic, but it's not. It's just the diff. I reviewed with git diff -W to confirm this.
As a non-disruptive, low-risk coding standards fix, this change is considered "rc eligible" and can be committed any time during RC. Reference: https://www.drupal.org/core/d8-allowed-changes#rc Committed and pushed to 8.0.x. Thanks!
Comments
Comment #2
David_Rothstein commentedComment #3
cilefen commentedI am tagging this "Novice" because a novice can provide a review on this coding-standards issue.
Comment #4
kikoalonsob commentedIt seems ok, "for" loop have the correct indentation now.
Comment #5
xjmThe diff generation here goofed me up and at first I thought this was changing the actual logic, but it's not. It's just the diff. I reviewed with
git diff -Wto confirm this.As a non-disruptive, low-risk coding standards fix, this change is considered "rc eligible" and can be committed any time during RC. Reference: https://www.drupal.org/core/d8-allowed-changes#rc Committed and pushed to 8.0.x. Thanks!