The code review bot has been updated today and now publishes all issues found on your patches as Phabricator lint results (Diff Detail section) instead of inline comments.
Here is a sample revision showcasing the new style. <https://phabricator-dev.allizom.org/D1758> (on phabricator staging) The main difference for developers is that lint results are tied to a specific patch; so when you update your revision, the lint results are automatically removed and only new ones are displayed (if any). Please be aware that any Error found by the bot must be fixed before landing, or your patch will break the CI. Warnings can still be ignored and should not break the build. The summary comment remains the same, and will always appear in the revision to keep track of the revision evolution. Thanks a lot to all the developers <https://bugzilla.mozilla.org/show_bug.cgi?id=1608339> who suggested this styling change, we hope you’ll enjoy this enhancement. If you have any questions regarding this change or the code review bot, you can reach us on Matrix #code-review-bot <https://chat.mozilla.org/#/room/#code-review-bot:mozilla.org>. Bastien Abadie _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
