Page MenuHomePhabricator

Fix the script that saves lint for a repo into the database and updates diffusion.
ClosedPublic

Authored by bluehawk on Mar 11 2014, 12:34 AM.
Tags
None
Referenced Files
Unknown Object (File)
May 9 2025, 5:07 AM
Unknown Object (File)
Apr 27 2025, 12:16 PM
Unknown Object (File)
Apr 18 2025, 11:20 AM
Unknown Object (File)
Apr 14 2025, 4:01 AM
Unknown Object (File)
Mar 24 2025, 12:19 PM
Unknown Object (File)
Mar 21 2025, 8:37 AM
Unknown Object (File)
Mar 12 2025, 5:27 AM
Unknown Object (File)
Mar 8 2025, 12:17 PM

Details

Summary

It appears a change to the way the configuration was loaded into ArcanistRepositoryAPI in rARCa2285b2b broke the save_lint script.
This updates the DiffusionLintSaveRunner to use the configuration correctly, allowing the linter to run

Test Plan

cd /your/project; ../../../path/to/phabricator/scripts/repository/save_lint.php

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

This revision is now accepted and ready to land.Mar 11 2014, 8:07 PM
epriestley updated this revision to Diff 20134.

Closed by commit rP8b6c86e27d14 (authored by @bluehawk, committed by @epriestley).