-
Notifications
You must be signed in to change notification settings - Fork 327
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
split jbake into core and application #274
Comments
Hello @ieugen, I share your opinion about multiple modules for jbake and I really like your poc :-). |
Thanks @danielgrycman . I just added watch functionality to 0.0.4 ( you net to specify I have alot of ideas but a lot of them need jbake refactoring. Right now I'm still waiting for answers but I am considering forking the project. I would very much like to avoid that and that's why I'm coming with issues and questions. |
Just posted a reply to your message on the users mailing list. Watch mode has already been added to master: #253 The plan to split up the project into multiple modules is available here: #117 - are you interested in working on this? If you want to start a discussion on the developers mailing list for your other ideas I can bring you up to speed with what is currently being worked on and by who? |
Hi, Thanks for the reply. I agree that things should be discussed and this is how comunity will grow. I think its time for action. My take from the discussions is:
I also think we should make the development/contribution process more open so the project will grow. I say this because we are all too busy to contribute from time to time and it is desirable that the project moves forward regardless of our personal schedules. I am a contributor to Apache Software Foundation, OPS4j, Debian Project and I know comunity is very important for an open source project. I would suggest jbake took some practices/policies/processes from those comunities. p.s. I would also like to have a chat (text or sound) with you @jonbullock to get to know eachother and the plans. |
👍 Thank you @ieugen for pushing the growing of the community forward. I'm looking forward to learn from your experience. |
@ieugen You are right the contribution process should be more open. If not so we loose the chance of a growing community :-). I would like to take part in this process :-). |
I realise there are multiple discussions going on in different places but for completeness sake I'll comment here: @ieugen could you provide some examples on practices/policies/processes you have in mind? I'll also try to be on IRC a bit more over the coming week so we can get a chat started. |
Linking this to the original issue #117 |
@jonbullock There are no silver bullets since each project is unique. This should also be done gradually, as the need arises and we should split to get benefits, not just because we can. The functionality that I see can be part of different modules right now is:
By separating them like this we allow people to embed jbake-core into their own applications, without dragging a lot of dependencies (like jetty and CLI parsing libs, etc). Am I making sense :)? |
@ieugen On the separation into sub modules, yeah absolutely. :) |
Hello,
I've posted an email on the user mailing list about some improvements.
Right now thare are a lot initializations that can be skipped and are not.
The code could make use of dependency injection to make the code easy to reuse.
It is also my opinion that jbake currently does way more than it should in a single project"
I think this should not be the case. I would split the code in multiple modules:
jbake-core (or a similar name ) module that does rendering
a jbake-cli (or jbakery) that uses the core and does command line parsing, web server and other stuff
Proof of concept project: https://github.com/netdava/jbakery
The text was updated successfully, but these errors were encountered: