diff --git a/kallithea/lib/hooks.py b/kallithea/lib/hooks.py
--- a/kallithea/lib/hooks.py
+++ b/kallithea/lib/hooks.py
@@ -157,10 +157,8 @@ def log_pull_action(ui, repo, **kwargs):
 
 def log_push_action(ui, repo, **kwargs):
     """
-    Maps user last push action to new changeset id, from mercurial
-
-    :param ui:
-    :param repo: repo object containing the `ui` object
+    Register that changes have been pushed.
+    Mercurial invokes this directly as a hook, git uses handle_git_receive.
     """
 
     ex = _extract_extras()
@@ -369,13 +367,13 @@ def log_delete_user(user_dict, deleted_b
     return 0
 
 
-handle_git_pre_receive = (lambda repo_path, revs, env:
-    handle_git_receive(repo_path, revs, env, hook_type='pre'))
-handle_git_post_receive = (lambda repo_path, revs, env:
-    handle_git_receive(repo_path, revs, env, hook_type='post'))
+def handle_git_pre_receive(repo_path, revs, env):
+    return handle_git_receive(repo_path, revs, env, hook_type='pre')
 
+def handle_git_post_receive(repo_path, revs, env):
+    return handle_git_receive(repo_path, revs, env, hook_type='post')
 
-def handle_git_receive(repo_path, revs, env, hook_type='post'):
+def handle_git_receive(repo_path, revs, env, hook_type):
     """
     A really hacky method that is run by git post-receive hook and logs
     an push action together with pushed revisions. It's executed by subprocess
diff --git a/kallithea/model/scm.py b/kallithea/model/scm.py
--- a/kallithea/model/scm.py
+++ b/kallithea/model/scm.py
@@ -505,6 +505,9 @@ class ScmModel(BaseModel):
                         % (scm_type,))
 
     def pull_changes(self, repo, username):
+        """
+        Pull from "clone URL".
+        """
         dbrepo = self.__get_repo(repo)
         clone_uri = dbrepo.clone_uri
         if not clone_uri:
@@ -536,10 +539,9 @@ class ScmModel(BaseModel):
     def commit_change(self, repo, repo_name, cs, user, author, message,
                       content, f_path):
         """
-        Commits changes
+        Commit a change to a single file
 
-        :param repo: SCM instance
-
+        :param repo: a db_repo.scm_instance
         """
         user = self._get_user(user)
         IMC = self._get_IMC_module(repo.alias)
@@ -677,6 +679,9 @@ class ScmModel(BaseModel):
 
     def update_nodes(self, user, repo, message, nodes, parent_cs=None,
                      author=None, trigger_push_hook=True):
+        """
+        Commits specified nodes to repo. Again.
+        """
         user = self._get_user(user)
         scm_instance = repo.scm_instance_no_cache()