Show a diff when a file will be or gets modified

[jgelens]

Now shows a diff of files which will be modified, only when its checksum its different.

Thanks @xvello (#985 (comment)) for the code!

This solves #985

• Pull request is based on the default branch (3.x at this time)
• Pull request includes tests for any new/updated operations/facts
• Pull request includes documentation for any new/updated operations/facts
• Tests pass (see scripts/dev-test.sh)
• Type checking & code style passes (see scripts/dev-lint.sh)

[jgelens]

Example output:

[jgelens]

Working on some improvements to not use `host.get_file'

[Fizzadar]

This is awesome, just noting I'm not forgetting it. Need to handle when to show/not show the diff (ie when -v). Going to get the pending stuff released and get then think on that to land this.

[jgelens]

Right, maybe it would be best to add a new flag for that. I always would like to see the diff, even when not using -v. So maybe we could add a --diff flag for this; similar to the same Ansible feature.

I was also thinking to add a global flag which can also per operation, in case people do not want to show it for e.g. editing a file with passwords.

[xvello]

I also think it's good to show the diffs without verbose logging, that's how I currently use it. A command line switch could indeed be enough.

@jgelens thanks for pushing this over the finish line. When you're pushing other changes, would you mind adding a Co-authored-by: Xavier Vello <xavier.vello@gmail.com> line in the git commit message of the first commit, for attribution?

[jgelens]

@jgelens thanks for pushing this over the finish line. When you're pushing other changes, would you mind adding a Co-authored-by: Xavier Vello <xavier.vello@gmail.com> line in the git commit message of the first commit, for attribution?

Of course, added.

[Fizzadar]

Right, maybe it would be best to add a new flag for that. I always would like to see the diff, even when not using -v. So maybe we could add a --diff flag for this; similar to the same Ansible feature.

👍 for --diff, good call.