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.

Sign up for GitHub

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

Jump to bottom

Refactor #6

Closed
Bigomby opened this issue Apr 21, 2016 · 0 comments
Closed

Refactor #6

Bigomby opened this issue Apr 21, 2016 · 0 comments
Assignees
@Bigomby
Labels
enhancement
Milestone
0.5

Comments

@Bigomby
Copy link
Contributor

Bigomby commented Apr 21, 2016 •
edited
Loading

The code needs to a refactor to remove dead code and some innecesary complexity.
@Bigomby Bigomby self-assigned this Apr 21, 2016
@Bigomby Bigomby added this to the 0.4 milestone Apr 21, 2016
@Bigomby Bigomby changed the title Refactorization Refactor Jun 10, 2016
@Bigomby Bigomby mentioned this issue Jul 22, 2016
Merged
5 tasks
@Bigomby Bigomby modified the milestones: 0.4, 0.5 Aug 2, 2016
@Bigomby Bigomby mentioned this issue Aug 2, 2016
Closed
@Bigomby Bigomby closed this as completed Aug 2, 2016
@Bigomby Bigomby reopened this Aug 2, 2016
@Bigomby Bigomby closed this as completed in 983b9da Sep 6, 2016
Bigomby added a commit that referenced this issue Oct 3, 2016
@Bigomby
Refactor (#8)
f565e0d 
Refactor the application to make it simplier.

The point is to have a minimal core and multiple reusable plugins to add functionality.

- No bidirectional references between structs.
- Remove logic that can be built into a plugin (limiter)
- Safe use of channels
- Use interfaces to hide implementation
- Loosely coupled modules
  - Reporter
  - Pipeline 

Closes #6
Bigomby added a commit that referenced this issue Oct 18, 2016
@Bigomby
Refactor (#8)
e58ef29 
Refactor the application to make it simplier.

The point is to have a minimal core and multiple reusable plugins to add functionality.

- No bidirectional references between structs.
- Remove logic that can be built into a plugin (limiter)
- Safe use of channels
- Use interfaces to hide implementation
- Loosely coupled modules
  - Reporter
  - Pipeline 

Closes #6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Bigomby Bigomby

Labels
enhancement
Projects
None yet
Milestone
0.5
Development

No branches or pull requests
1 participant
@Bigomby