Merge lp://staging/~cjwatson/codetree/default-branch into lp://staging/codetree
Proposed by
Colin Watson
| Status: | Merged |
|---|---|
| Approved by: | Tom Haddon |
| Approved revision: | 125 |
| Merged at revision: | 125 |
| Proposed branch: | lp://staging/~cjwatson/codetree/default-branch |
| Merge into: | lp://staging/codetree |
| Diff against target: |
37 lines (+15/-4) 1 file modified
codetree/handlers/git.py (+15/-4) |
| To merge this branch: | bzr merge lp://staging/~cjwatson/codetree/default-branch |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Tom Haddon (community) | Approve | ||
| Canonical IS Reviewers | Pending | ||
|
Review via email:
|
|||
Commit message
Use remote default branch rather than hardcoding master
Description of the change
A repository's default branch might be called something other than "master" (for example, "main" is becoming popular), and it would make more sense to prefer the default branch if none is specified.
This handles a problem that came up in https:/
To post a comment you must log in.
Revision history for this message
| 🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote | # |
Revision history for this message
| 🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
This merge proposal is being monitored by mergebot. Change the status to Approved to merge.