Page MenuHomePhabricator

Add ref lookup for username symbols
ClosedPublic

Authored by epriestley on Apr 12 2020, 3:05 PM.
Tags
None
Referenced Files
F15337661: D21092.diff
Sun, Mar 9, 7:01 AM
Unknown Object (File)
Fri, Feb 21, 7:04 PM
Unknown Object (File)
Feb 9 2025, 8:54 AM
Unknown Object (File)
Feb 9 2025, 8:54 AM
Unknown Object (File)
Feb 9 2025, 8:54 AM
Unknown Object (File)
Jan 28 2025, 5:36 AM
Unknown Object (File)
Jan 25 2025, 6:59 AM
Unknown Object (File)
Jan 25 2025, 6:59 AM
Subscribers
None

Details

Summary

Ref T13490. I'm attempting to update "arc amend", but it needs to fetch revision authors to raise an "amending a revision you don't own" error.

Support user-symbol resolution.

Along the way, this introduces some infrastructure for abstracting away iteration over a multi-page Conduit result set.

Test Plan
  • Ran "arc inspect ..." for various "user(...)" queries.
  • Set page size to 3 and issued a general query, saw the future page it away properly.

Diff Detail

Repository
rARC Arcanist
Branch
aflow7
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24053
Build 33114: Run Core Tests
Build 33113: arc lint + arc unit

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Apr 12 2020, 8:42 PM
This revision was automatically updated to reflect the committed changes.