-
Notifications
You must be signed in to change notification settings - Fork 91
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
Github flow #51
Comments
Hi Jordi, |
Hello @ondovb There is no problem at all in having in working it in a separated repo. It is true that we do not have to worry about pull requests then but I will relay on you in modifying the C++/C API. If the idea is to have mashpy totally separated I agree on it but there are things like the CMake building process, the C++ tests, the CI Server that should be in the main project. What do you think about me doing all this work in my own fork of the project and if we see that it is useful for the main project ending merging some of the features? |
Yes, I think the fork makes sense for now, and I can merge things when I get time to look closer. |
Now that more than one person is going to contribute to the project and more than one change is going to be done would be great to have a clear github-flow.
The standard in many projects is to keep two branches: Master and Development. The master is equivalent to the releases meanwhile the development goes ahead with the last stable features in the project. Each time that a functionality has to be added a new branch is created from development and they merge when the feature is working.
I have no permissions so @ondovb should create it.
Does it seem right for you?
The text was updated successfully, but these errors were encountered: