Skip to content

Config load with require#125

Merged
SanderElias merged 4 commits intomasterfrom
configLoadWithRequire
Dec 31, 2019
Merged

Config load with require#125
SanderElias merged 4 commits intomasterfrom
configLoadWithRequire

Conversation

@SanderElias
Copy link
Copy Markdown
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • [x ] Other... Please describe:
    Improves ease of plugins

What is the current behavior?

Issue Number: N/A

What is the new behavior?

get rid of the protective context to make registering plugins easier.
Also, add an cmd-line option for alternate config files (--cf)
also, add on cmd-line option to show the browser (--sb)

Does this PR introduce a breaking change?

  • Yes
  • [ x] No

@SanderElias SanderElias merged commit 4dd239a into master Dec 31, 2019
@SanderElias SanderElias deleted the configLoadWithRequire branch December 31, 2019 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants