Change TODOs format.
Use TODO: instead of TODO
This commit is contained in:
@@ -21,11 +21,11 @@ def main():
|
|||||||
elif os.path.isfile(config_path):
|
elif os.path.isfile(config_path):
|
||||||
config = load_config(config_path)
|
config = load_config(config_path)
|
||||||
else:
|
else:
|
||||||
# TODO replace the next line with something to do if no config file is found
|
# TODO: replace the next line with something to do if no config file is found
|
||||||
config = load_config(config_path)
|
config = load_config(config_path)
|
||||||
pass
|
pass
|
||||||
|
|
||||||
# TODO make the command line arguments work and override the config options
|
# TODO: make the command line arguments work and override the config options
|
||||||
|
|
||||||
synchroniser = Synchroniser(
|
synchroniser = Synchroniser(
|
||||||
config.roots.remote,
|
config.roots.remote,
|
||||||
|
|||||||
@@ -14,7 +14,7 @@ def unisync_sync(synchroniser:Synchroniser, paths_manager:PathsManager):
|
|||||||
synchroniser.sync_files(paths_manager.get_paths_to_sync())
|
synchroniser.sync_files(paths_manager.get_paths_to_sync())
|
||||||
synchroniser.sync_links(paths_manager.get_paths_to_sync())
|
synchroniser.sync_links(paths_manager.get_paths_to_sync())
|
||||||
|
|
||||||
# TODO check the config options and do or don't do the following
|
# TODO: check the config options and do or don't do the following
|
||||||
synchroniser.update_links()
|
synchroniser.update_links()
|
||||||
#synchroniser.mount_remote_dir()
|
#synchroniser.mount_remote_dir()
|
||||||
|
|
||||||
|
|||||||
@@ -75,7 +75,6 @@ class Synchroniser:
|
|||||||
On slow connections one might want to increase this.
|
On slow connections one might want to increase this.
|
||||||
Returns:
|
Returns:
|
||||||
An error code (0 success, 1 TimeoutExpired, 2 KeyboardInterrupt).
|
An error code (0 success, 1 TimeoutExpired, 2 KeyboardInterrupt).
|
||||||
TODO change that to raising the exception.
|
|
||||||
"""
|
"""
|
||||||
self.control_path = os.path.expanduser(control_path)
|
self.control_path = os.path.expanduser(control_path)
|
||||||
command = [
|
command = [
|
||||||
@@ -87,6 +86,7 @@ class Synchroniser:
|
|||||||
"-p", str(self.remote_port)
|
"-p", str(self.remote_port)
|
||||||
]
|
]
|
||||||
master_ssh = subprocess.Popen(command)
|
master_ssh = subprocess.Popen(command)
|
||||||
|
# TODO: Raise an exception instead of changing the return value
|
||||||
try:
|
try:
|
||||||
ret_code = master_ssh.wait(timeout=connection_timeout)
|
ret_code = master_ssh.wait(timeout=connection_timeout)
|
||||||
except subprocess.TimeoutExpired:
|
except subprocess.TimeoutExpired:
|
||||||
|
|||||||
Reference in New Issue
Block a user