[luigi.file] removed (spotify#2832)

* luigi.file removed * master branch in travis
drowoseque · Nov 26, 2019 · 4fbdb4a · 4fbdb4a
1 parent d108eb5
commit 4fbdb4a
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 33 deletions.
diff --git a/luigi/contrib/ftp.py b/luigi/contrib/ftp.py
@@ -33,7 +33,7 @@
 import io
 
 import luigi
-import luigi.file
+import luigi.local_target
 import luigi.format
 import luigi.target
 from luigi.format import FileWrapper

diff --git a/luigi/contrib/opener.py b/luigi/contrib/opener.py
@@ -31,7 +31,7 @@
 
 import json
 
-from luigi.file import LocalTarget
+from luigi.local_target import LocalTarget
 from luigi.mock import MockTarget
 from luigi.contrib.s3 import S3Target
 from luigi.target import FileSystemException

diff --git a/luigi/file.py b/luigi/file.py
diff --git a/luigi/target.py b/luigi/target.py
@@ -189,7 +189,7 @@ def copy(self, path, dest):
 
 class FileSystemTarget(Target):
     """
-    Base class for FileSystem Targets like :class:`~luigi.file.LocalTarget` and :class:`~luigi.contrib.hdfs.HdfsTarget`.
+    Base class for FileSystem Targets like :class:`~luigi.local_target.LocalTarget` and :class:`~luigi.contrib.hdfs.HdfsTarget`.
 
     A FileSystemTarget has an associated :py:class:`FileSystem` to which certain operations can be
     delegated. By default, :py:meth:`exists` and :py:meth:`remove` are delegated to the
@@ -313,7 +313,7 @@ class AtomicLocalFile(io.BufferedWriter):
     moving it to its final destination.
 
     This class is just for the writing part of the Target. See
-    :class:`luigi.file.LocalTarget` for example
+    :class:`luigi.local_target.LocalTarget` for example
     """
 
     def __init__(self, path):

diff --git a/test/contrib/opener_test.py b/test/contrib/opener_test.py
@@ -6,7 +6,7 @@
 
 from luigi.contrib.opener import OpenerTarget, NoOpenerError
 from luigi.mock import MockTarget
-from luigi.file import LocalTarget
+from luigi.local_target import LocalTarget
 
 from nose.plugins.attrib import attr
 
@@ -79,8 +79,8 @@ def test_local_target(self):
         target.open('w').close()
         self.assertTrue(LocalTarget.fs.exists(self.local_file))
 
-    @mock.patch('luigi.file.LocalTarget.__init__')
-    @mock.patch('luigi.file.LocalTarget.__del__')
+    @mock.patch('luigi.local_target.LocalTarget.__init__')
+    @mock.patch('luigi.local_target.LocalTarget.__del__')
     def test_local_tmp_target(self, lt_del_patch, lt_init_patch):
         '''Verify local target url with query string