launchpad git config instructions should include your username
Bug #1600055 reported by
Thomi Richards
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Low
|
Colin Watson | ||
Bug Description
On a project's +configure-code page, in the git setup instructions, the instructions mention:
```
Push a new repository
You can add a remote for your Git repository with the command:
git remote add origin git+ssh:
… and push the Git branch to Launchpad with:
git push origin master
```
However this will fail if the user's local shell username is different from their launchpad username. Ideally the instructions should render the users launchpad username, like so:
```
git remote add origin git+ssh://<email address hidden>/pmr
```
Related branches
lp:~blr/launchpad/bug-1600055-configure-code-remote-username
- Colin Watson (community): Approve
-
Diff: 232 lines (+93/-83)4 files modifiedlib/lp/code/templates/configure-code-macros.pt (+1/-1)
lib/lp/registry/browser/product.py (+11/-0)
lib/lp/registry/browser/tests/test_product.py (+81/-0)
lib/lp/registry/browser/tests/test_product_views.py (+0/-82)
lp:~cjwatson/launchpad/git-ssh-url-username
- Maximiliano Bertacchini (community): Approve
- Launchpad code reviewers: Pending requested
-
Diff: 149 lines (+40/-12)5 files modifiedlib/lp/code/browser/gitref.py (+13/-1)
lib/lp/code/browser/gitrepository.py (+13/-1)
lib/lp/code/browser/tests/test_gitref.py (+4/-3)
lib/lp/code/browser/tests/test_gitrepository.py (+8/-5)
lib/lp/code/templates/git-macros.pt (+2/-2)
| Changed in launchpad: | |
| assignee: | nobody → Kit Randel (blr) |
| importance: | Undecided → Medium |
| Changed in launchpad: | |
| status: | New → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| William Grant (wgrant) wrote | #2 |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → In Progress |
Revision history for this message
| Maximiliano Bertacchini (maxiberta) wrote | #3 |
Revision history for this message
| api.ng (hektve) wrote | #4 |
| Changed in launchpad: | |
| status: | In Progress → Fix Released |
| assignee: | Kit Randel (blr) → nobody |
| assignee: | nobody → HECTOR DAVID (hektve) |
| Changed in launchpad: | |
| assignee: | HECTOR DAVID (hektve) → nobody |
| status: | Fix Released → Triaged |
| Changed in launchpad: | |
| status: | Triaged → In Progress |
| assignee: | nobody → Colin Watson (cjwatson) |
| importance: | Medium → Low |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #5 |
| tags: |
added: qa-needstesting removed: qa-ok |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r18152 <http://