tl;dr: The Code Review Bot will now warn you about CI-breaking issues on Phabricator
Hello, We have just released a new version of the code review bot that changes slightly how lint issues are reported on Phabricator: - Issues breaking the CI are now reported as Phabricator lint results. Those are unignorable, contrary to inline comments. We are highlighting them because, if landed, such patches would be backed-out by the sheriffs. They are displayed at the top of the revision, and in the patch. They also are automatically removed on new diffs. - Ignorable lint issues (warnings) and static analysis results are still reported as inline comments. This will help you know when your patch has issues which would cause a back-out. You can view a demonstration of such an error report here: https://phabricator.services.mozilla.com/D59862 Bastien & Marco _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
