Skip to content
This repository has been archived by the owner on Apr 6, 2021. It is now read-only.

loadConfig should accept array of config, not just file path #71

Open
matmar10 opened this issue Feb 12, 2019 · 0 comments
Open

loadConfig should accept array of config, not just file path #71

matmar10 opened this issue Feb 12, 2019 · 0 comments

Comments

@matmar10
Copy link

Currently, loadConfig only accepts a file path. However, an equally reasonable approach is to pass the array of plugin definitions directly to loadConfig.

loadConfig supports a JS file, but it requires synchronous loading and return of the plugin list directly via module.exports, which may not be feasible in all applications.

Proposed change adds the following capability:

loadConfig(configArray, base, [callback])

  • configArray array of plugins to load with their configuration
  • base location to use as the base for plugins
  • callback an (optional) callback

PR inbound.

matmar10 added a commit to blossomfinance/architect that referenced this issue Feb 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant