Excellent idea Corsix.. makes sense considering if multiple changes are made by many people then you'd want to know the author.
Further, I would STRONGLY RECOMMEND a thread that shows the following:
WHO MADE THE CHANGE
WHAT FILE WAS CHANGED
WHAT SECTIONS WERE CHANGED
WHAT THOSE CHANGES DID
WHEN DO THESE NEW SCRIPTS COMMENCE
WHO WAS IMPACTED (ALL or SPECIFIC RACE)
Looking good!
Replying to File editing conventions
Topic Summary
thudo
Posted 18 January 2005 - 01:17 AM
Grey
Posted 18 January 2005 - 12:20 AM
I like it.
Will give us a basic form of version control.
Of course we'd have to agree on in advance what to work solo on.
Will give us a basic form of version control.
Of course we'd have to agree on in advance what to work solo on.
Corsix
Posted 17 January 2005 - 08:45 PM
I propose a new standard for when you edit/create AI files:
Edited files:At top of file:
Created files:At top of file:
What you do guys think about this?
Edited files:At top of file:
---------------------------------------- -- File: 'FILEPATH/FILENAME.ai' -- Edited by Person1 @ 01.01.2005 -- Edited by Person2 @ 16.01.2005 -- Edited by Person1 @ 28.01.2005 -- Edited by Person3 @ 08.03.2005for example:
---------------------------------------- -- File: 'ai/cpu_manager.ai' -- Edited by Thudmeizer @ 01.01.2005 -- Edited by Corsix @ 16.01.2005edited/added sections:
-- SomePerson Modded/Added CODE --EndFor example:
--Corsix Added function BuildBaseStrategy:AddOnNotify( addon_id, notify_code , build_channel) aitrace("this race does not have an addon notify!") end --End
--Corsix Modded local id_2 = self.post_addon_2_id -- Added these two lines local before = math.abs(self.percent_post_upgraded - (num_up_post/num_post*100)) local after = math.abs(self.percent_post_upgraded - ((num_up_post+1)/num_post*100)) --Added "and"... + changed plan type if( self:PlanExists("Build NotifiedAddOn Plan", id_2) == false ) and after <= before then self.AddPlan( self, BuildNotifiedAddOnPlan( id_2 , 0) ) end local id_2 = self.turret_addon_2_id -- Added these two lines local before = math.abs(self.percent_turr_upgraded - (num_up_turr/num_turr*100)) local after = math.abs(self.percent_turr_upgraded - ((num_up_turr+1)/num_turr*100)) --Added "and"... + changed plan type if( self:PlanExists("Build NotifiedAddOn Plan", id_2) == false ) and after <= before then self.AddPlan( self, BuildNotifiedAddOnPlan( id_2 , 1) ) end --End
Created files:At top of file:
---------------------------------------- -- File: 'FILEPATH/FILENAME.ai' -- Created by Person1 @ 01.01.2005 -- Edited by Person2 @ 16.01.2005 -- Edited by Person1 @ 28.01.2005 -- Edited by Person3 @ 08.03.2005for example:
---------------------------------------- -- File: 'ai/plans/buildnotifiedaddonplan.ai' -- Created by Corsix @ 17.01.2005
What you do guys think about this?