safer 4.12.2__tar.gz → 5.0.0__tar.gz

{safer-4.12.2 → safer-5.0.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: safer
-Version: 4.12.2
+Version: 5.0.0
 Summary: 🧿 A safer writer for files and streams 🧿
 Home-page: https://github.com/rec/safer
 License: MIT
@@ -14,7 +14,6 @@ Classifier: Programming Language :: Python :: 3.9
 Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
-Requires-Dist: black (>=23.12.0,<24.0.0)
 Project-URL: Documentation, https://rec.github.io/safer
 Project-URL: Repository, https://github.com/rec/safer
 Description-Content-Type: text/markdown

{safer-4.12.2 → safer-5.0.0}/pyproject.toml

@@ -1,14 +1,6 @@
-[tool.black]
-line-length = 88
-skip-string-normalization = true
-target-version = ["py38"]
-
-[tool.doks]
-auto = true
-
 [tool.poetry]
 name = "safer"
-version = "4.12.2"
+version = "5.0.0"
 description = "🧿 A safer writer for files and streams 🧿"
 authors = ["Tom Ritchford <tom@swirly.com>"]
 license = "MIT"
@@ -19,17 +11,14 @@ documentation = "https://rec.github.io/safer"
 
 [tool.poetry.dependencies]
 python = ">=3.8"
-black = "^23.12.0"
 
 [tool.poetry.group.dev.dependencies]
 coverage = "*"
-flake8 = "*"
 pytest = "*"
 readme-renderer = "*"
 tdir = "*"
 toml = "*"
 pyyaml = "*"
-isort = "*"
 ruff = "*"
 mypy = "*"
 
@@ -46,6 +35,18 @@ exclude_lines = [
     "if __name__ == .__main__.:",
     "raise NotImplementedError"
 ]
+
+[tool.ruff]
+line-length = 88
+
+[tool.ruff.format]
+quote-style = "single"
+
+
+[tool.mypy]
 [build-system]
 requires = ["poetry-core"]
 build-backend = "poetry.core.masonry.api"
+
+[tool.doks]
+auto = true

{safer-4.12.2 → safer-5.0.0}/safer/__init__.py

@@ -1,5 +1,4 @@
-"""
-# 🧿 `safer`: A safer writer 🧿
+"""# 🧿 `safer`: A safer writer 🧿
 
 Avoid partial writes or corruption!
 
@@ -42,6 +41,10 @@ https://pypi.org/project/atomicwrites/
 It also has a useful `dry_run` setting to let you test your code without
 actually overwriting the target file.
 
+NOTE: Just like plain old `open`, if a file that is already opened for writing
+is opened again before the first write has completed, the results are
+unpredictable: so don't do it!
+
 * `safer.writer()` wraps an existing writer, socket or stream and writes a
   whole response or nothing
 
@@ -67,7 +70,6 @@ writes the data to a temporary file on disk, which is moved over using
 does not work on Windows.  (In fact, it's unclear if any of this works on
 Windows, but that certainly won't.  Windows developer solicted!)
 
-
 ### Example: `safer.writer()`
 
 `safer.writer()` wraps an existing stream - a writer, socket, or callback -
@@ -143,6 +145,7 @@ With `safer`
         for item in items:
             print(item)
         # Either the whole file is written, or nothing
+
 """
 import contextlib
 import functools
@@ -419,13 +422,13 @@ def open(
 
     if is_binary:
         if 't' in mode:
-            raise ValueError('can\'t have text and binary mode at once')
+            raise ValueError("can't have text and binary mode at once")
         if newline:
-            raise ValueError('binary mode doesn\'t take a newline argument')
+            raise ValueError("binary mode doesn't take a newline argument")
         if encoding:
-            raise ValueError('binary mode doesn\'t take an encoding argument')
+            raise ValueError("binary mode doesn't take an encoding argument")
         if errors:
-            raise ValueError('binary mode doesn\'t take an errors argument')
+            raise ValueError("binary mode doesn't take an errors argument")
 
     if 'x' in mode and os.path.exists(name):
         raise FileExistsError("File exists: '%s'" % name)
@@ -657,14 +660,16 @@ class _FileRenameCloser(_FileCloser):
         self.target_file = target_file
         self.dry_run = dry_run
         self.is_binary = is_binary
+        if temp_file is True:
+            parent, file = os.path.split(target_file)
+            temp_file = os.path.join(parent, f'.{file}.tmp-safer')
+
         super().__init__(temp_file, delete_failures, parent)
 
     def _success(self):
         if not self.dry_run:
             if os.path.exists(self.target_file):
                 shutil.copymode(self.target_file, self.temp_file)
-            else:
-                os.chmod(self.temp_file, 0o100644)
             os.replace(self.temp_file, self.target_file)
 
         elif callable(self.dry_run):