
from __future__ import absolute_import

from zope.interface import Interface


class IFilesystemSnapshots(Interface):
    """
    Support creating and listing snapshots of a specific filesystem.
    """
    def create(name):
        """
        Create a snapshot of the filesystem.

        :param name: The name of the snapshot.
        :type name: :py:class:`flocker.snapshots.SnapshotName`

        :return: Deferred that fires on snapshot creation, or errbacks if
            snapshotting failed. The Deferred should support cancellation
            if at all possible.
        """


    def list():
        """
        Return all the filesystem's snapshots.

        :return: Deferred that fires with a ``list`` of
            :py:class:`flocker.snapshots.SnapshotName`. This will likely be
            improved in later iterations.
        """


from __future__ import absolute_import

from zope.interface import Interface


class IFilesystemSnapshots(Interface):
    """
    Support creating and listing snapshots of a specific filesystem.
    """
    def create(name):
        """
        Create a snapshot of the filesystem.

        :param name: The name of the snapshot.
        :type name: :py:class:`flocker.snapshots.SnapshotName`

        :return: Deferred that fires on snapshot creation, or errbacks if
            snapshotting failed. The Deferred should support cancellation
            if at all possible.
        """


    def list():
        """
        Return all the filesystem's snapshots.

        :return: Deferred that fires with a ``list`` of
            :py:class:`flocker.snapshots.SnapshotName`.
        """

Address review comment: Don't state the obvious.
Address review comment: Don't state the obvious.

  
  from __future__ import absolute_import
  
  from zope.interface import Interface
  
  
  class IFilesystemSnapshots(Interface):
      """
      Support creating and listing snapshots of a specific filesystem.
      """
      def create(name):
          """
          Create a snapshot of the filesystem.
  
          :param name: The name of the snapshot.
          :type name: :py:class:`flocker.snapshots.SnapshotName`
  
          :return: Deferred that fires on snapshot creation, or errbacks if
              snapshotting failed. The Deferred should support cancellation
              if at all possible.
          """
  
  
      def list():
          """
          Return all the filesystem's snapshots.
  
          :return: Deferred that fires with a ``list`` of
-             :py:class:`flocker.snapshots.SnapshotName`. This will likely be
+             :py:class:`flocker.snapshots.SnapshotName`.
-             improved in later iterations.
          """
  
Address review comment: Don't state the obvious.
## Code Before:

from __future__ import absolute_import

from zope.interface import Interface


class IFilesystemSnapshots(Interface):
    """
    Support creating and listing snapshots of a specific filesystem.
    """
    def create(name):
        """
        Create a snapshot of the filesystem.

        :param name: The name of the snapshot.
        :type name: :py:class:`flocker.snapshots.SnapshotName`

        :return: Deferred that fires on snapshot creation, or errbacks if
            snapshotting failed. The Deferred should support cancellation
            if at all possible.
        """


    def list():
        """
        Return all the filesystem's snapshots.

        :return: Deferred that fires with a ``list`` of
            :py:class:`flocker.snapshots.SnapshotName`. This will likely be
            improved in later iterations.
        """

## Instruction:
Address review comment: Don't state the obvious.
## Code After:

from __future__ import absolute_import

from zope.interface import Interface


class IFilesystemSnapshots(Interface):
    """
    Support creating and listing snapshots of a specific filesystem.
    """
    def create(name):
        """
        Create a snapshot of the filesystem.

        :param name: The name of the snapshot.
        :type name: :py:class:`flocker.snapshots.SnapshotName`

        :return: Deferred that fires on snapshot creation, or errbacks if
            snapshotting failed. The Deferred should support cancellation
            if at all possible.
        """


    def list():
        """
        Return all the filesystem's snapshots.

        :return: Deferred that fires with a ``list`` of
            :py:class:`flocker.snapshots.SnapshotName`.
        """
