Merge lp:~mars/tarmac/fix-plugin-import into lp:~launchpad/tarmac/lp-tarmac
Proposed by
Māris Fogels
Status: | Merged |
---|---|
Approved by: | Māris Fogels |
Approved revision: | 384 |
Merged at revision: | 385 |
Proposed branch: | lp:~mars/tarmac/fix-plugin-import |
Merge into: | lp:~launchpad/tarmac/lp-tarmac |
Diff against target: |
112 lines (+43/-6) 3 files modified
tarmac/plugin.py (+5/-1) tarmac/plugins/allowedcontributors.py (+23/-3) tarmac/plugins/tests/test_allowedcontributors.py (+15/-2) |
To merge this branch: | bzr merge lp:~mars/tarmac/fix-plugin-import |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Māris Fogels (community) | Approve | ||
Review via email:
|
Commit message
Fix the Tarmac plugin activation code so plugins in .config are recognized.
Description of the change
Hi,
This branch presents a quick and dirty fix to the Tarmac plugin activation code. It adds the Tarmac plugin directories to the system path so that Tarmac can import the plugins.
There may be a better way to do this, but I am leaving that better implementation for someone else to tackle. The code passed manual testing on my local system.
Maris
To post a comment you must log in.
Self-approved for the Launchpad Tarmac trial and landing on the lp-tarmac fork.