Merge lp:~rockstar/launchpad/code-js-file-reorg into lp:launchpad
Proposed by
Paul Hummer
Status: | Merged |
---|---|
Approved by: | Michael Hudson-Doyle |
Approved revision: | no longer in the source branch. |
Merged at revision: | not available |
Proposed branch: | lp:~rockstar/launchpad/code-js-file-reorg |
Merge into: | lp:launchpad |
Prerequisite: | lp:~rockstar/launchpad/code-js-reorg |
Diff against target: |
244 lines (+44/-45) 10 files modified
lib/canonical/launchpad/javascript/code/branch.bugspeclinks.js (+2/-2) lib/canonical/launchpad/javascript/code/branch.status.js (+3/-4) lib/canonical/launchpad/javascript/code/branchmergeproposal.diff.js (+3/-3) lib/canonical/launchpad/javascript/code/branchmergeproposal.reviewcomment.js (+3/-3) lib/canonical/launchpad/javascript/code/branchmergeproposal.status.js (+2/-2) lib/lp/app/templates/base-layout-macros.pt (+17/-4) lib/lp/bugs/templates/bugtask-index.pt (+2/-2) lib/lp/code/templates/branch-index.pt (+4/-18) lib/lp/code/templates/branch-related-bugs-specs.pt (+2/-2) lib/lp/code/templates/branchmergeproposal-index.pt (+6/-5) |
To merge this branch: | bzr merge lp:~rockstar/launchpad/code-js-file-reorg |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michael Hudson-Doyle | Approve | ||
Review via email: mp+21085@code.launchpad.net |
Description of the change
In the early days of javascript, no sheriff meant no law. No law meant crime
was at an all time high.
This branch basically removes some of our javascript crimes. Specifically,
when we created javascript, we named the file and namespace whatever seemed to
work. When everyone has their own ideas on how this works, it means we have a
great amount of inconsistency. I changed this by naming the javascript files
to be <model>
needed to change some of the YUI namespaces.
All the windmill tests pass, etc.
To post a comment you must log in.
Looks suitably boring. Is the code going to move into lib/lp/code at some point?