Page MenuHomePhabricator

List branch on main repository view
ClosedPublic

Authored by chad on Aug 17 2017, 6:30 PM.
Tags
None
Referenced Files
F15405114: D18441.diff
Tue, Mar 18, 10:19 AM
F15331312: D18441.diff
Fri, Mar 7, 8:51 AM
Unknown Object (File)
Thu, Feb 20, 10:40 AM
Unknown Object (File)
Thu, Feb 20, 10:39 AM
Unknown Object (File)
Thu, Feb 20, 10:36 AM
Unknown Object (File)
Wed, Feb 19, 4:05 AM
Unknown Object (File)
Feb 16 2025, 11:44 PM
Unknown Object (File)
Feb 16 2025, 11:08 PM
Subscribers

Details

Summary

This is in the crumbs, but a little hidden. Puts branch name at the top of the browse table header.

Test Plan

Review a few branchs, change branch, see new name.

Diff Detail

Repository
rP Phabricator
Branch
branch-name-header (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18098
Build 24319: Run Core Tests
Build 24318: arc lint + arc unit

Event Timeline

epriestley added inline comments.
src/applications/diffusion/controller/DiffusionRepositoryController.php
570

Does not function correctly for popular and reasonable branch name "0".

570–572

Weird indentation.

This revision is now accepted and ready to land.Aug 17 2017, 6:50 PM
This revision was automatically updated to reflect the committed changes.