copy: pylint and docs (#30847)

This PR includes;
- pylint fixes
- Documentation updates
This commit is contained in:
Dag Wieers 2017-10-30 00:52:45 +01:00 committed by GitHub
commit a6415f9ad9
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -55,7 +55,7 @@ options:
if the destination does not exist. if the destination does not exist.
type: bool type: bool
default: 'yes' default: 'yes'
aliases: [ "thirsty" ] aliases: [ thirsty ]
version_added: "1.1" version_added: "1.1"
directory_mode: directory_mode:
description: description:
@ -88,8 +88,8 @@ extends_documentation_fragment:
- validate - validate
- decrypt - decrypt
author: author:
- "Ansible Core Team" - Ansible Core Team
- "Michael DeHaan" - Michael DeHaan
notes: notes:
- The M(copy) module recursively copy facility does not scale to lots (>hundreds) of files. - The M(copy) module recursively copy facility does not scale to lots (>hundreds) of files.
For alternative, see M(synchronize) module, which is a wrapper around C(rsync). For alternative, see M(synchronize) module, which is a wrapper around C(rsync).
@ -252,7 +252,7 @@ def adjust_recursive_directory_permissions(pre_existing_dir, new_directory_list,
Walk the new directories list and make sure that permissions are as we would expect Walk the new directories list and make sure that permissions are as we would expect
''' '''
if len(new_directory_list) > 0: if new_directory_list:
working_dir = os.path.join(pre_existing_dir, new_directory_list.pop(0)) working_dir = os.path.join(pre_existing_dir, new_directory_list.pop(0))
directory_args['path'] = working_dir directory_args['path'] = working_dir
changed = module.set_fs_attributes_if_different(directory_args, changed) changed = module.set_fs_attributes_if_different(directory_args, changed)