[13:18:20] dbrant: mdholloway: I ran a format on the project and pushed the patch. There were a lot of unformatted xmls, so the changes are huge, but there will be no impact [ other than that all to-be-merged patches will have to rebase]. If you guys feel comfortable can we please merge right away? [13:25:44] sharvani: sure. i'm giving it a look for the sake of due diligence but it looks fine (after the couple of checkstyle violations are cleared up). i'm assuming no functional changes are introduced. [13:27:54] Thanks mdholloway: No functional changes . And will fix the checkstyle [13:29:57] by the way, it looks like the gerrit reviewer auto-tagging bot is down at the moment; this happens from time to time. i try to keep an eye on the patches for the entire project (not just the ones i'm tagged on) but it's probably worth double-checking to ensure that reviewers are added to your patches after pushing (and adding them yourself if not) [13:30:02] i'll add us all to this patch [13:42:47] Oh will keep an eye out for that . Thanks Michael ! I saw your comments on the patch . Will fix that in a bit .