Comments about default branch selection

Rafael Manzo · Heitor
1 parent 466a15b4
Showing 1 changed file with 3 additions and 1 deletions Show diff stats
app/assets/javascripts/repository/branch.js.coffee
@@ -15,10 +15,12 @@ class Repository.Branch
       @request.abort()
       @request = null
  
-  # private method
   fill_options: (options, el) ->
+    # FIXME: this works only if master is the default branch
+    #        it can be improved by moving it into KalibroProcessor
     default_branch = "master"
     if default_branch in options
+      # Brings the default branch as the first option
       index = options.indexOf(default_branch)
       options.splice(index, 1)
       options.unshift(default_branch)
...	...	@@ -15,10 +15,12 @@ class Repository.Branch
15	15	@request.abort()
16	16	@request = null
17	17
18		- # private method
19	18	fill_options: (options, el) ->
	19	+ # FIXME: this works only if master is the default branch
	20	+ # it can be improved by moving it into KalibroProcessor
20	21	default_branch = "master"
21	22	if default_branch in options
	23	+ # Brings the default branch as the first option
22	24	index = options.indexOf(default_branch)
23	25	options.splice(index, 1)
24	26	options.unshift(default_branch)
...	...