Project

General

Profile

Bug #3870

add a directory for modules to add config in

Added by Greg Sutcliffe almost 6 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Category:
Hammer core
Target version:
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Instead of:

:modules:
- hammer_cli_foreman
etc

it would be nice to have something like $confdir/hammer.modules.d/ where a modules could add config of their own (e.g. enable foreman module by creating $confdir/hammer.modules.d/hammer_cli_foreman). This makes it much easier to enable modules from separate OS packages (as they can just drop a file in place, instead of sed'ing the main hammer config file)


Related issues

Related to Hammer CLI - Bug #5052: Documentation "Writing you own plugin" needs updateClosed2014-04-03

Associated revisions

Revision b73e2544 (diff)
Added by Martin Bacovsky over 5 years ago

Fixes #3870 - add a directory for module's configs

Revision f73b380e
Added by mbacovsky over 5 years ago

Merge pull request #92 from mbacovsky/3870_modules_config

Fixes #3870 - add a directory for module's configs

History

#1 Updated by Dmitri Dolguikh over 5 years ago

  • Target version set to 1.9.0

#2 Updated by Martin Bacovsky over 5 years ago

  • Assignee set to Martin Bacovsky
  • Category set to Hammer core
  • Status changed from New to Assigned

#3 Updated by Martin Bacovsky over 5 years ago

  • Status changed from Assigned to Ready For Testing

#4 Updated by Martin Bacovsky over 5 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Applied in changeset foreman:hammer-cli|commit:b73e2544052fae7e739be260a8d4fd1364a4f539.

#5 Updated by Tomáš Strachota over 5 years ago

  • Related to Bug #5052: Documentation "Writing you own plugin" needs update added

Also available in: Atom PDF