merge: tag '1.1.0' into develop

Better error handling when merging files and user interaction in this phase

Author: ElanDeyan

.gitignore

@@ -1,5 +1,6 @@
 data/
 build/
 dist/
+.spec
 *.pdf
 *.pyc

README.md

@@ -7,19 +7,19 @@ This is a basic project to merge pdf files.
 To get help, run:
 
 ```sh
-path/to/exe -h
+python -m pdf_merger -h
 ```
 
 You can use it like this:
 
 ```sh
-path/to/exe -d dir/with/pdf/files -o result/file.pdf
+python -m pdf_merger -d dir/with/pdf/files -o result/file.pdf
 ```
 
 Or using the long form:
 
 ```sh
-path/to/exe --directory dir/with/pdf/files --output result/file.pdf
+python -m pdf_merger --dir dir/with/pdf/files --output result/file.pdf
 ```
 
 > [!NOTE]

changelog.md

@@ -0,0 +1,23 @@
+## **1.1.0**&emsp;<sub><sup>2024-07-20 (e37d7b607f61e70b27caf17aeb0918eea4db0ee5...5121c8390c5637cded459375869982d4bbfdee9d)</sup></sub>
+
+### Features
+
+- first version (e49d7ce4751fbaecef63cda4388eeb392ea01e40)
+- better error handling and user interaction when merge (c6191f25fbc199ad97dca496c1bbf178dfd24c20)
+
+### Bug Fixes
+
+- corrects the long form of the directory argument in example and command in examples (5121c8390c5637cded459375869982d4bbfdee9d)
+
+### Merges
+
+- branch 'release/1\.0\.0' (a4dea194f2faeea0d78fccad0f68601401a51f82)
+- tag '1\.0\.0' into develop (f0f5b8f992140cc68a556d6294e1ff2fb476c9dd)
+
+### ? ? ?
+
+- Initial commit (e37d7b607f61e70b27caf17aeb0918eea4db0ee5)
+
+<br>
+
+

pdf_merger.py

@@ -21,12 +21,25 @@ def pdf_files_in_directory(path: Path) -> Optional[list[Path]]:
 
 
 def merge_pdf_files(files: list[Path], result_path: Path):
-    writer = PdfWriter()
+    try:
+        writer = PdfWriter()
 
-    for file in files:
-        writer.merge(None, fileobj=PdfReader(file, True))
+        for file in files:
+            writer.merge(None, fileobj=PdfReader(file, True))
 
-    writer.write(result_path.with_suffix(PDF_SUFFIX))
+        if result_path.with_suffix(PDF_SUFFIX).exists():
+            response = input(
+                f"The file {result_path.name}.pdf exists, do you want to overwrite? (y/N): "
+            )
+
+            if response.lower() != "y":
+                print("Operation canceled.")
+                return
+
+        writer.write(result_path.with_suffix(PDF_SUFFIX))
+        print('PDFs merged succesfully!')
+    except Exception as e:
+        print(f"Failed to merge: {e}")
 
 
 def main():