Repository: ansible/ansible

Issue:
Documentation for the module import_role require small corrections
##### ISSUE TYPE
 - Documentation Report

##### COMPONENT NAME
import_role

##### ANSIBLE VERSION
<!--- Paste verbatim output from "ansible --version" between quotes below -->
```
ansible 2.4.1.0
  config file = /etc/ansible/ansible.cfg
  configured module search path = [u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python2.7/site-packages/ansible
  executable location = /usr/bin/ansible
  python version = 2.7.14 (default, Nov  3 2017, 10:55:25) [GCC 7.2.1 20170915 (Red Hat 7.2.1-2)]
```

##### CONFIGURATION
Nothing changed in configuration.

##### OS / ENVIRONMENT
N/A

##### SUMMARY
The documentation for the role import_role states:

> Most keyworkds, loops and conditionals will not be applied to this action, but to the imported tasks instead. If you want the opposite behaviour, use import_role instead.

It's recursion to itself. Probably another rule name should be used instead in the part of the sentence "use import_rule instead".

Additionally, a typo exists in the word "keyworkds" - should be "keywords".

I'm not sure about rules used in English in Ansible documentation - means should be used American English or British English. If the first one the word "behaviour" should be changed to "behavior" accordingly (?).

##### EXPECTED RESULTS
Name of another rule in the reference.


Discussion:
Files identified in the description:
* [lib/ansible/modules/utilities/logic/import_role.py](https://github.com/ansible/ansible/blob/devel/lib/ansible/modules/utilities/logic/import_role.py)

If these files are inaccurate, please update the `component name` section of the description or use the `!component` bot command.

[click here for bot help](https://github.com/ansible/ansibullbot/blob/master/ISSUE_HELP.md)
<!--- boilerplate: components_banner --->
A part of [the file to change](https://github.com/ansible/ansible/blob/4617be30859e043bf50513cbcc04be4c922d37e7/lib/ansible/modules/utilities/logic/import_role.py#L15-L24)
Hi,
Yup, it's a bit confusing. If you have a suggestion of better wording could you please raise a PR to update it. Docs should be American English.
I'm bit confused now because in the code is the correct module name referenced. But when I display help using ansible-doc I see.
![ansible-doc import_role_001](https://user-images.githubusercontent.com/4910261/33236990-d6c3920c-d265-11e7-8540-2fbdf3639865.jpg)

I've installed Ansible using an rpm package for Fedora Linux

```
> rpm -qi ansible            

Name        : ansible
Version     : 2.4.1.0
Release     : 2.fc26
Architecture: noarch
Install Date: Thu 09 Nov 2017 06:17:41 AM CET
Group       : Development/Libraries
Size        : 40236248
License     : GPLv3+
Signature   : RSA/SHA256, Tue 31 Oct 2017 04:37:50 PM CET, Key ID 812a6b4b64dab85d
Source RPM  : ansible-2.4.1.0-2.fc26.src.rpm
Build Date  : Tue 31 Oct 2017 01:17:27 AM CET
Build Host  : buildhw-08.phx2.fedoraproject.org
Relocations : (not relocatable)
Packager    : Fedora Project
Vendor      : Fedora Project
URL         : http://ansible.com
Summary     : SSH-based configuration management, deployment, and task execution system
Description :

Ansible is a radically simple model-driven configuration management,
multi-node deployment, and remote task execution system. Ansible works
over SSH and does not require any software or daemons to be installed
on remote nodes. Extension modules can be written in any language and
are transferred to managed machines automatically.
```

Resolving patch:
diff --git a/lib/ansible/modules/utilities/logic/import_role.py b/lib/ansible/modules/utilities/logic/import_role.py
--- a/lib/ansible/modules/utilities/logic/import_role.py
+++ b/lib/ansible/modules/utilities/logic/import_role.py
@@ -23,7 +23,8 @@
   - Much like the `roles:` keyword, this task loads a role, but it allows you to control it when the role tasks run in
     between other tasks of the play.
   - Most keywords, loops and conditionals will only be applied to the imported tasks, not to this statement itself. If
-    you want the opposite behaviour, use M(include_role) instead.
+    you want the opposite behavior, use M(include_role) instead. To better understand the difference you can read
+    U(https://docs.ansible.com/ansible/latest/playbooks_reuse_includes.html).
 version_added: "2.4"
 options:
   name:
@@ -63,7 +64,7 @@
 - hosts: all
   tasks:
     - import_role:
-       name: myrole
+        name: myrole
 
     - name: Run tasks/other.yaml instead of 'main'
       import_role:


Test patch: