Merge lp:~rharding/launchpad/remove_mustache into lp:launchpad
Proposed by
Richard Harding
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Aaron Bentley | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 15487 | ||||
Proposed branch: | lp:~rharding/launchpad/remove_mustache | ||||
Merge into: | lp:launchpad | ||||
To merge this branch: | bzr merge lp:~rharding/launchpad/remove_mustache | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Aaron Bentley (community) | Approve | ||
Review via email: mp+111633@code.launchpad.net |
Commit message
Remove mustache.js from the sourcedeps list.
Description of the change
Per the bug, we've subsumed mustache into lib/app/
There is no need to pull the mustache dep into the sourcecode directory. This change just removes that from the config.
To post a comment you must log in.
I don't think it's a good idea to fork mustache.js, and I've filed bug #1016668 about it. However, with the current state of the code, removing the sourcecode line does no harm.