project_issue_file_review 6.x-2.8-rc4

Added in 6.x-2.8-rc1:

• #1395712 by jthorson: Exclude external files from core coder reviews
• #1338148 by rfay: Remove call time pass-by-reference instances from pfir_client.review.inc
• #1346010 by jthorson: Remove -p0 patch detection attempt
• #1347588 by jthorson: Fix undefined index error if no dependencies
• #1347590 by jthorson: fix undefined debug index in watchdog messages
• #1347594 by jthorson: Only list relevant test cases in review log
• #1233248 by jthorson: check for empty --file argument before executing run_tests.sh, check output for FATALs
• #1289502 by jthorson: Find tests in places other than module root and /test directories
• Issues #1410768, #1410688 by jthorson: Added testbot id and timestamp to review log
• by jthorson: Fixed typo in previous commit
• #639232 by webchick, boombatower: Fix plurality in summary message
• #1352220 by jthorson: Support 'langcode' install argument for D8 compatibility
• #1347582 by jthorson: Delay clear_vars() op until after xmlrpc call completes
• #1361468 by jthorson: Modified pifr_server 'all environment' result check

Added in 6.x-2.8-rc2:

• Item #1395712 by jthorson: Changed Symfony exclude to core/vendor directory
• #1361468 by jthorson: Fixed typo in earlier patch (all env result check)

Added in 6.x-2.8-rc3:

• #1411974 by jthorson: Add shortcut testbot confirmation option
• #1443180 by Albert Volkman: grammer fix
• #1436412 by jthorson: Fix for 'unable to clear checkout directory'
• #1347594 by jthorson: Remove verbose logging during successful exec calls
• #1437268 by jthorson: Modified windows line ending check
• #1361468 by jthorson: Prevent duplicate pifr_environment_status entry warnings

Added in 6.x-2.8-rc4:

• #1437268 by jthorson: Update server/confirmation/check.patch after changing line ending check
• #1436412 by jthorson: Removed error from failed chmod command
• #1436412 by jthorson: Typo in previous commit