Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cleanup entry #47

Merged
merged 8 commits into from Oct 20, 2016
Merged

Cleanup entry #47

merged 8 commits into from Oct 20, 2016

Conversation

kilahm
Copy link
Member

@kilahm kilahm commented Oct 18, 2016

  • Separate instantiation of systems from hooking them up via events.
  • Make creation of the main HackUnit instance depend on Options, but nothing else.

This should make it easier to introduce new CLI options, and to allow a config file to be merged with the cli options.

This paves the way for individual projects to instantiate their own
versions of portions of the project if desired.
This redesign of the entry point will hopefully allow for an easier
addition of cli options.
Toss in a fix to the valid suite fixture.
@kilahm kilahm merged commit cca8393 into develop Oct 20, 2016
@kilahm kilahm deleted the cleanup-entry branch October 20, 2016 00:12
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.

None yet

1 participant