search

traindb-project / traindb-ml

Remote ML Model Serving Component for TrainDB
Apache License 2.0
6 stars 2 forks source link

Branching rule #20

Closed kihyuk-nam closed 2 years ago

kihyuk-nam commented 2 years ago

I hope the main branch be clean. By clean I mean all the codes achieved consensus among participants. How about the following branching rule?

  1. write codes on personal branches
  2. PR the codes that need to be merged
  3. everyone reviews the 'PR'ed codes (add comments)
  4. merge them only after reaching an agreement

@sungsoo @1jolee

sungsoo commented 2 years ago

I agree with your opinion. So I forked the repository for personal workspace. I'll send the merge request for the main traindb-ml repository if the merge is needed.

Thank you.