Merge branch 'commit-count' into 'master'

Project commit count for default branch Currently, the commit count displayed on a project's home page shows the total number of commits on the master branch. This can lead to situations in which a project may have no commits reported or very few if the project workflow does not involve the use of a branch named "master". Since number of commits can be an indicator of project health, showing 0 commits could make it harder to encourage adoption of a project. This MR alters the repository model's `commit_count` method to return the count for the default branch instead of master. See issue #299

Merge branch 'commit-count' into 'master'
Project commit count for default branch Currently, the commit count displayed on a project's home page shows the total number of commits on the master branch. This can lead to situations in which a project may have no commits reported or very few if the project workflow does not involve the use of a branch named "master". Since number of commits can be an indicator of project health, showing 0 commits could make it harder to encourage adoption of a project. This MR alters the repository model's `commit_count` method to return the count for the default branch instead of master. See issue #299
Dmitriy Zaporozhets
2 parents 8e5a0b6e 62e523f4
Showing 1 changed file with 1 additions and 1 deletions Show diff stats
app/models/repository.rb
@@ -106,7 +106,7 @@ class Repository
   def commit_count
     Rails.cache.fetch(cache_key(:commit_count)) do
       begin
-        raw_repository.raw.commit_count
+        raw_repository.raw.commit_count(self.root_ref)
       rescue
         0
       end
...	...	@@ -106,7 +106,7 @@ class Repository
106	106	def commit_count
107	107	Rails.cache.fetch(cache_key(:commit_count)) do
108	108	begin
109		- raw_repository.raw.commit_count
	109	+ raw_repository.raw.commit_count(self.root_ref)
110	110	rescue
111	111	0
112	112	end
...	...