FlightGear Git: splitting FGData: Difference between revisions

Some answers for HHS's questions
m (Forgot to add my name)
(Some answers for HHS's questions)
Line 95: Line 95:
But with that there are lot of rules with exceptions. From my daily real life job I do know, that many rules with exceptions may make things more complicated and provoke discussions and conflicts.
But with that there are lot of rules with exceptions. From my daily real life job I do know, that many rules with exceptions may make things more complicated and provoke discussions and conflicts.
So my question is: How can we see that "a reasonable level of competence in both aircraft development and GIT usage" is there? Does he has to be checked? Which level of competence is needed for making an aircraft?
So my question is: How can we see that "a reasonable level of competence in both aircraft development and GIT usage" is there? Does he has to be checked? Which level of competence is needed for making an aircraft?
(some answers from TorstenD)
I'd define "reasonable level of competence" for GIT as "be able be familiar with the basic everyday GIT workflow". If [http://cs.swan.ac.uk/~csoliver/ok-sat-library/internet_html/doc/doc/Git/1.7.4.1/Documentation/everyday.html everyday git] is understandable, I'd say OK.
And for aircraft development: Be able to stay in your sandbox, don't mess with other's files, be able to reuse existing (common) files, create reasonable file sizes, accept naming conventions etc. Most important: nobody is perfect, but be able to fix what you broke ;-)


(hvengel comments)  
(hvengel comments)  
Line 103: Line 108:


(hvengel)
(hvengel)


It is good to see that the existing rules are finally formalized, so we get a clear guideline.  
It is good to see that the existing rules are finally formalized, so we get a clear guideline.  
It is also good to see that people should be more encouraged to submit merge requests. But this also means that those with commit rights should be more encouraged to review those merge requests and commit them!
It is also good to see that people should be more encouraged to submit merge requests. But this also means that those with commit rights should be more encouraged to review those merge requests and commit them!