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

Remove extended functionality #92

Closed
henrikbjorn opened this issue Dec 18, 2013 · 6 comments
Closed

Remove extended functionality #92

henrikbjorn opened this issue Dec 18, 2013 · 6 comments

Comments

@henrikbjorn
Copy link
Contributor

The Core of Bernard have gotten quite fat with require-dev dependencies. Theese should be kept to a minimum for testing and the logger.

@sagikazarmark
Copy link
Contributor

Hm, what do you mean? Split the drivers and the core to different packages?
I would also place the driver dependencies in the suggest part of composer.json with a brief description what they are needed for.

@henrikbjorn
Copy link
Contributor Author

Yup

So the Producer / Consumer part is one thing and each driver is seperate.

@sagikazarmark
Copy link
Contributor

Maintaining core would be obviously easier, but having each driver in a separate package...that's weird and make it harder to use. Not an easy question IMO.

@henrikbjorn
Copy link
Contributor Author

the more drivers the longer the require for development and ci becomes, also it requires new releases of bernard instead of individual driver releases.

@sagikazarmark
Copy link
Contributor

You are right. Probably a list of pros and cons could clear the problem.

@sagikazarmark
Copy link
Contributor

Duplicate of #256 and #258

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

No branches or pull requests

2 participants