New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update known hosts #3

Open

aded wants to merge 4 commits into JulienVdG:master from aded:update-known-hosts

Contributor

aded commented Nov 19, 2018

No description provided.

aded added 4 commits

November 19, 2018 16:17


          Porting to Ansible 2.7: include_tasks and import_tasks can no longer …

…accept inline variables


          No need to report changes when trying to establish ssh connection

41451d5


          Merge branch 'no-changes-in-ssh-connection'

b0465a2


          Update local known_hosts

db89e5c

Owner

JulienVdG commented Nov 20, 2018

I've not looked into the details yet but at first glance:

this is a new feature, it's lacking documentation, and probably an option to disable it (and tests, but my current tests are so bad...)
you don't have a license header in your update_known_hosts.yml file (should be apache 2.0 as the rest)
you are changing tasks/main.yml you should add your name/copyright and a short description of the change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet