-
Notifications
You must be signed in to change notification settings - Fork 1
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
Document in the wiki the process and conventions that are to be used to maintain the standard #39
Comments
For standard development work that we are doing I think we are more after the BRANCH option
From: Niels Klazenga [mailto:Niels.Klazenga@rbg.vic.gov.au] And that I know how to do, as that is how I edited the AVH Advanced Search page. Also, anyone can do that, not just us, and we still get to decide what goes in and what not. I think both viable alternatives.
Were you suggesting we each work in our own branch? Or was it that you thought we could share a development branch? GitHub has the concept of forking, which goes even further, copying the entire tree into my account, perhaps this is even better? Cheers, Ben |
I think whether to fork or not to fork is not the question. People who are not owners who do want to make changes have to fork anyway and have to do a pull request. Owners can decide for themselves whether they want to push directly into the hiscom/hispid repository, or want to push to a fork they created in their own account and send a pull request, which they can then merge themselves (seems an unnecessary detour to me).. The question is what we want to do with changes that are pushed or pulled into the hiscom/hispid repository, whether they go straight into the master, or whether they go first into a branch, which can later be merged into master. Aaron wants the latter and I think (now) that makes sense, as that means the master will never be in between two "versions" and will always have the current version. |
Yes - that is a good summary Niels. |
need to document conventions for:
branching
milestones
The text was updated successfully, but these errors were encountered: