This updates everywhere to use the history table to pull in lint and coverage information. At the moment the history entry is still resolved via the branch identifier.
This is not yet complete (i.e. it is a draft)
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Maniphest Tasks
- T5001: Store Diffusion lint information against a commit hash instead of a branch
N/A
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- historical-1
- Lint
Lint Errors Severity Location Code Message Error src/applications/diffusion/controller/DiffusionLintController.php:54 XHP5 Use of Undeclared Variable Error src/applications/diffusion/request/DiffusionRequest.php:365 PHL1 Unknown Symbol Warning resources/sql/autopatches/20140515.history.2.sql:1 TXT3 Line Too Long Warning resources/sql/autopatches/20140515.history.2.sql:3 TXT3 Line Too Long Advice src/applications/diffusion/DiffusionLintSaveRunner.php:130 XHP16 TODO Comment Advice src/applications/diffusion/request/DiffusionRequest.php:355 XHP16 TODO Comment - Unit
Tests Passed - Build Status
Buildable 473 Build 473: [Placeholder Plan] Wait for 30 Seconds