Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Maniphest Tasks
- T7604: Remove "Arcanist Projects"
- Commits
- Restricted Diffusion Commit
rP021223907dfd: Remove arcanist projects from differential
Submitted a diff. Patched the diff with arc patch.
Diff Detail
- Repository
- rP Phabricator
- Branch
- arpj2
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 6066 Build 6086: [Placeholder Plan] Wait for 30 Seconds
Event Timeline
I'm guessing it is okay to remove the differential.getrevision and differential.getdiff endpoints? They were deprecated some time ago.
If they don't make this change harder (I think they don't), I'd very slightly prefer to leave them until T5873 / T7715. I think those are happening soon (next few months) and I'm currently looking at compressing as many breaking API changes into a small window of time as I can in connection with them (basically, make a single major revision to the API instead of breaking it a bunch of times over the next few months).
Particularly, if we remove them today, anything relying on them will need to switch to revision.query, and then be broken again by T7715. If we get rid of them both at once, clients can do a single update.