resolving suggested fixed

Author: codeperfectplus

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,23 +1,16 @@
 
 # Pull Request Template
 
-**script name** -
-
 **What have you Changed**
 
 what you changed in the codebase.write here
 
 
-### Issue no.(must) - #
+### Issue No. closes #
 
 ### Self Check(Tick After Making pull Request)
 
-- [ ] This issue was assigned to me.
-- [x] One Change in one Pull Request
-- [x] My file is in proper folder
-- [x] I am following clean code and Documentation and my code is well linted with flake8.
-- [ ] I have added README.md and requirements.txt with my script
-
-If issue was not assigned to you Please don't make a PR. It will marked as invalid.
+- [ ] One Change in one Pull Request
+- [ ] I am following clean code and Documentation and my code is well linted with flake8.
 
 Join Us on Discord:- https://discord.gg/JfbK3bS

CHANGELOG.md

@@ -2,7 +2,28 @@
 
 All notable changes to this project will be documented in this file. The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [Initial Release]
+## [1.0.5] - 02-11-2022
+
+- PYPI package is now available
+- Only create required directories
+- Package can be updated from Github action workflow
+
+## [1.0.4] - 04-07-2022
+
+- code refactoring
+- bug fixes
+
+## [1.0.3] - 23-09-2020
+
+- Bug Fixed
+
+## [1.0.2] - 22-09-2020
+
+- New Extension Added
+- Bug Fixed
+
+
+## [Initial Release] - 13/05/2020
 
 - Initial Release
 

filemover/main.py

@@ -56,9 +56,11 @@ def move_files(folder_path:str, file_folder_map: dict):
         ext_file_map (dict) : File to Folder map
     '''
     for folder, files in file_folder_map.items():
+        os.makedirs(os.path.join(folder_path, folder), exist_ok=True)
         for file in files:
-            os.makedirs(os.path.join(folder_path, folder), exist_ok=True)
-            move(os.path.join(folder_path, file), os.path.join(folder_path, folder))
+            old_file_path = os.path.join(folder_path, file)
+            new_file_path = os.path.join(folder_path, folder)
+            move(old_file_path, new_file_path)
 
 
 def get_folder(ext):
@@ -77,7 +79,8 @@ def get_folder(ext):
     return 'Other'
 
 
-def start(folder_path: str): # need to change function name
+# TODO  need to change function name
+def start(folder_path: str):    
     '''
     Organize files on the current directory, each to the corresponding folder.