Page MenuHomePhabricator
Differential D12734

Remove the repositories application
ClosedPublic
Actions

Authored by joshuaspence on May 6 2015, 12:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 10, 6:29 AM
Unknown Object (File)
Mon, May 5, 6:13 PM
Unknown Object (File)
Sun, May 4, 3:26 AM
Unknown Object (File)
Sun, May 4, 2:44 AM
Unknown Object (File)
Fri, Apr 25, 6:45 AM
Unknown Object (File)
Sun, Apr 20, 12:48 PM
Unknown Object (File)
Fri, Apr 18, 2:02 PM
Unknown Object (File)
Apr 10 2025, 12:01 AM
View All Files
Subscribers
epriestley
Tokens
"Doubloon" token, awarded by epriestley.

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T7603: Remove the deprecated "Repositories" application
Commits
Restricted Diffusion Commit
rPc4bcfc85f160: Remove the repositories application
Summary

Fixes T7603. This application is no longer used.

Test Plan

Went to /repository/ and got a 404.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 30597.May 6 2015, 12:47 PM
joshuaspence retitled this revision from to [Draft] Remove the repositories application.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a task: T7603: Remove the deprecated "Repositories" application.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 6 2015, 12:47 PM
Herald added a subscriber: epriestley. · View Herald Transcript
epriestley accepted this revision.May 17 2015, 2:53 PM
epriestley added a reviewer: epriestley.

This (or some substantially equivalent future patch) is desirable when dependencies are resolved.

This revision is now accepted and ready to land.May 17 2015, 2:53 PM
joshuaspence added a comment.May 25 2015, 9:30 PM

Just confirming... I think this is basically safe to land now?

joshuaspence updated this revision to Diff 31382.May 25 2015, 9:33 PM
joshuaspence edited edge metadata.

Rebase

Herald added a subscriber: Korvin. · View Herald TranscriptMay 25 2015, 9:33 PM
joshuaspence retitled this revision from [Draft] Remove the repositories application to Remove the repositories application.May 25 2015, 9:33 PM
joshuaspence updated this object.
joshuaspence removed a subscriber: Korvin.
joshuaspence edited the test plan for this revision. (Show Details)May 26 2015, 11:58 AM
joshuaspence added a comment.Jun 1 2015, 5:36 AM

I'm going to land this now... the Repositories application doesn't serve a purpose any longer.

Closed by commit rPc4bcfc85f160: Remove the repositories application (authored by joshuaspence, committed by joshuaspence). · Explain WhyJun 1 2015, 5:40 AM
This revision was automatically updated to reflect the committed changes.
epriestley awarded a token.Jun 1 2015, 1:47 PM

Revision Contents
Changeset List

Diff 31601

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/repository/application/PhabricatorRepositoriesApplication.php

Loading...
View Options

src/applications/repository/controller/PhabricatorRepositoryController.php

Loading...
View Options

src/applications/repository/controller/PhabricatorRepositoryListController.php

Loading...
Phabricator · Built by Phacility