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

Authored by bluehawk on Mar 11 2014, 12:34 AM.

Tags

None

Referenced Files

	F15515522: D8487.id20106.diff
	Fri, Apr 18, 11:20 AM

	F15502761: D8487.id20134.diff
	Mon, Apr 14, 4:01 AM

	F15431440: D8487.id.diff
	Mar 24 2025, 12:19 PM

	F15419751: D8487.id20106.diff
	Mar 21 2025, 8:37 AM

	F15369118: D8487.id20134.diff
	Mar 12 2025, 5:27 AM

	F15334801: D8487.id20134.diff
	Mar 8 2025, 12:17 PM

	F15282977: D8487.diff
	Mar 4 2025, 4:05 AM

	Unknown Object (File)
	Feb 14 2025, 10:54 PM

Subscribers

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

Restricted Diffusion Commit
rP8b6c86e27d14: Fix the script that saves lint for a repo into the database and updates…

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

Thanks!

This revision is now accepted and ready to land.Mar 11 2014, 8:07 PM

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

Revision Contents
Changeset List

Path

Size

src/

applications/

diffusion/

DiffusionLintSaveRunner.php

5 lines

Diff 20134

View Options

src/applications/diffusion/DiffusionLintSaveRunner.php

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