From a465324cba01769cccdb5ea2386487905eb84a6c Mon Sep 17 00:00:00 2001
From: Eric Torres <erictorres4@protonmail.com>
Date: Mon, 15 Apr 2019 23:48:47 -0700
Subject: [PATCH] Docstring updates

---
 rbackup/plugins/packagemanager.py | 19 ++++++++++---------
 rbackup/struct/hierarchy.py       |  2 +-
 rbackup/struct/repository.py      |  5 +++--
 3 files changed, 14 insertions(+), 12 deletions(-)

diff --git a/rbackup/plugins/packagemanager.py b/rbackup/plugins/packagemanager.py
index bfe5db0..f78d47c 100644
--- a/rbackup/plugins/packagemanager.py
+++ b/rbackup/plugins/packagemanager.py
@@ -24,8 +24,7 @@ class PackageManager:
 
     The package manager can be used in conjunction with a ``Snapshot`` for backups.
 
-    Lockfile Management
-    ^^^^^^^^^^^^^^^^^^^
+    **Lockfile Management**
 
     This class can be used as a context manager for creating a lockfile for the
     specific package manager. This is to prevent transactions from occurring during
@@ -34,6 +33,9 @@ class PackageManager:
 
         .. note:: Subclasses can override the context manager and implement i.e. blocking until
             the process is complete with a timeout.
+
+        .. note:: The lockfile is only created if it was configured. Otherwise it is silently
+            ignored.
     """
 
     def __init__(self, cachedir=None, db_path=None, lockfile=None, pkglist_cmd=None):
@@ -63,6 +65,8 @@ class PackageManager:
             either there is an ongoing transaction in progress or a previous transaction
             failed and the database is in an inconsistent state.
 
+        This method only creates the lockfile if it was configured.
+
         :returns: self
         :rtype: ``PackageManager`` object
         :raises FileExistsError: if lockfile exists when this method is called
@@ -107,9 +111,6 @@ class PackageManager:
     def gen_db_archive(self, compress=None):
         """Generate a database archive for this package manager.
 
-            All arguments and keyword-only arguments are passed directly
-            to the PackageManager object.
-
         :param compress: compression mode
         :type compress: str
         :returns: the path to the created file
@@ -142,7 +143,7 @@ class PackageManager:
     def cache_directory(self):
         """
         :returns: the cache directory of this package manager
-        :rtype: path-like object
+        :rtype: path-like object or None
         """
         return self._cachedir
 
@@ -150,7 +151,7 @@ class PackageManager:
     def database_path(self):
         """
         :returns: the database path of this package manager
-        :rtype: path-like object
+        :rtype: path-like object or None
         """
         return self._db_path
 
@@ -158,7 +159,7 @@ class PackageManager:
     def lockfile(self):
         """
         :returns: the lockfile path of this package manager
-        :rtype: path-like object
+        :rtype: path-like object or None
         """
         return self._lockfile
 
@@ -166,6 +167,6 @@ class PackageManager:
     def pkglist_cmd(self):
         """
         :returns: the package listing command of this package manager
-        :rtype: iterable or str
+        :rtype: iterable, str or None
         """
         return self._pkglist_cmd
diff --git a/rbackup/struct/hierarchy.py b/rbackup/struct/hierarchy.py
index ef40be1..5e4a33e 100644
--- a/rbackup/struct/hierarchy.py
+++ b/rbackup/struct/hierarchy.py
@@ -26,7 +26,7 @@ class Hierarchy(PathLike):
     to call either :func:`shutil.mkdir` or related method to create
     the directory structure it emulates.
 
-    Implementation Details
+    **Implementation Details**
 
     * For consistency, ``Hierarchy`` objects always store and return absolute paths
     * Data for all ``Hierarchy`` objects and subclassed objects use JSON for serialization
diff --git a/rbackup/struct/repository.py b/rbackup/struct/repository.py
index 1d703d6..b458c67 100644
--- a/rbackup/struct/repository.py
+++ b/rbackup/struct/repository.py
@@ -5,9 +5,10 @@
 
 A repository is a directory that contains backup data
 sequestered into snapshots and a symlink to the most
-recently created snapshot.
+recently created snapshot. Additionally, it has a metadata
+dot file for the names of the snapshots.
 
-Properties
+**Properties**
 
 * Each snapshot in a repository is unaware of one another,
     this is the job of the repository to organize