Differential D15300

Update import/clear symbols scripts for callsigns
ClosedPublic
Actions

Authored by epriestley on Feb 18 2016, 11:13 AM.

Details

Reviewers

chad

Maniphest Tasks

T4245: Make repository URIs slightly prettier

Commits

Restricted Diffusion Commit
rP78fba426f661: Update import/clear symbols scripts for callsigns

Summary

Ref T4245. Accept identifiers instead of callsigns in these scripts so things continue to work in a future callsign-optional world.

Test Plan

Ran these scripts with both valid and invalid arguments; saw success and errors.

Diff Detail

Repository

rP Phabricator

Branch

call6

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 10750
Build 13228: Run Core Tests
Build 13227: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 36907.Feb 18 2016, 11:13 AM

epriestley retitled this revision from to Update import/clear symbols scripts for callsigns.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: chad.

epriestley added a task: T4245: Make repository URIs slightly prettier.

noidea

This revision is now accepted and ready to land.Feb 18 2016, 4:13 PM

Closed by commit rP78fba426f661: Update import/clear symbols scripts for callsigns (authored by epriestley, committed by epriestley). · Explain WhyFeb 18 2016, 5:56 PM

This revision was automatically updated to reflect the committed changes.

• Persona added a child revision: D15341: Fix Ponder Exception, spacing.Feb 24 2016, 11:16 AM

tycho.tatitscheff removed a child revision: D15341: Fix Ponder Exception, spacing.Feb 24 2016, 12:31 PM

Revision Contents
Changeset List

Path

Size

scripts/

symbols/

clear_repository_symbols.php

16 lines

import_repository_symbols.php

16 lines

Diff 36907

View Options

scripts/symbols/clear_repository_symbols.php