-
-
Notifications
You must be signed in to change notification settings - Fork 13
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
create a plan for moving generator out [Main task] #100
Comments
@wahaj-47 what is your take on it? |
@EbrahimKreem @wahaj-47 it's not a pending task right now - but maybe we should start a conversation about it. tell me what your opinion/questions about it |
Are we moving the generator to a separate repo ? |
@wahaj-47 not right now. But I'm thinking about it.
tell me your questions! it will also help us to not change our logic frequently, and we'll also be able to move tests, that we have to generator outside of this module as well. |
@Edebo @vadim9999 what is your take on it? |
if generator can stand alone,i think we should go ahead.considering the listed advantages.i just hope there wont be side effects.would love to hear @vadim9999 take |
i think that main advantage will be less complicated code and whole functionality. When I decided to split fetch into 2 modules - i didn't plan to add here our team members - this is why repository name is so stupid. But with our current progress - i definately want to do it. Also! if it's interested - i can elaborate more about microservices |
only have a rough idea of microservices. am interested in the elaborate part.then we should separate generator then if we are going for this approach |
microservices is just one of the patterns. main idea is - separate functionality as much as possible. another simple example:
the most simple service is one separated function that is multilanguage, so you can connect different languages to in simultaneously |
get it. |
@Edebo please ask me questions in order to provoke me to update task description |
this will be a repository for our generator. i'll clean it up later, but right now that setup is similar to sd repository. So our code will be validated and published to npm without any issues(at least with less of them :) |
@Edebo imagine that tomorrow I'll assign you a task about moving out the generator - tell me your 5-10 questions that will help you do it quickly/without errors |
1)What are the steps to follow |
|
@Edebo let's think about questions from another perspective. (still, I asked about 5-10 questions for a reason) I think both questions/situations can require to spend 1 hour for each - because planning is an important thing, especially in this case. So another imaginable situation: |
@atherdon,i agree with planning being important.
for the imaginable situation: |
will put everything at one place
Yes we are. Few times. This is why I decide to move this functionality away from here - because for the first time it was just a simple script. But it evolves, mature and right now it just takes a lot of "space" at this repo. Another point is to make it more advanced - via separation. because separation will create some unpredictable cases - and it will help us to improve code again. Process of separation is simple - we need to put everything - related to generator into that separated repo. we need to move code/tests/sub-modules. then it should be published at npm and installed into sd repo. then we need to call functions from generator module at sd code scripts or at pkg.json commands.
1)create workable solution for the process.(i think if we have a solution for the process breaking down wont be difficult) we also should handle tests: https://github.com/GroceriStar/sd/blob/master/tests/projects/ Plus we should add tests coverage for checking all methods that we have at generator(excerpt projects folder) and when generator will be moved out - we should remove explanation of methods from Readme and put a link to new generator readme - at separated repo [not finished] |
Readme also have a great explanation of methods. And that part should be moved to a readme at separated repo as well https://github.com/GroceriStar/sd#generator-commands |
am thinking of these steps- these are coming from the things you talked about 1)let us adjust the generator methods-what should be achieved in the adjustment |
1) please explain what exact methods we should move?
Plus i think we'll have option "fix bug if any exist" after each of this
points :)
… |
i was refering to
|
We should create steps about what files should be moved. |
@vadim9999 i agree. partially we discuss it at this long thread. maybe you can put everything at one text so it will be easy to read to all of us? |
@vadim9999 maybe we should put all information at this readme - https://github.com/GroceriStar/sd/tree/master/generator |
ok, I will do it |
@vadim9999 we have a new member @svr8 and i think you can combine your forces and work together on moving generator out. |
@vadim9999 quiestion: do you think it'll be more comfortable for you to work on this module, if'll migrate all tasks, related to generator to the separated repository issues? |
yes it will be good |
I'm on it
… |
@Edebo am i right that you're back? do you have pending tasks that you need to complete? |
@atherdon am done with #471 ,while the pending task is what @vadim9999 is working on |
can you add a link to that task as well?
… |
got it - then you're free and can help with tasks that located at: https://github.com/GroceriStar/food-static-files-generator |
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Is it was created as task at GitHub issues? Please provide an issue number or link
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: