Update linting (#90)

* update * update * update tests and fix ci * update lint * update readme * update requirements * update linting
redhat-cop · Mar 13, 2023 · 3f4378a · 3f4378a
1 parent 0b649c1
commit 3f4378a
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 0 deletions.
diff --git a/changelogs/fragments/.gitkeep b/changelogs/fragments/.gitkeep
diff --git a/changelogs/fragments/changed_when_lint.yml b/changelogs/fragments/changed_when_lint.yml
@@ -0,0 +1,4 @@
+---
+minor_changes:
+  - Added changed_when false to built in commands that gather information.
+...
diff --git a/roles/virtualenv_migrate/tasks/00_gather_custom_requirements.yml b/roles/virtualenv_migrate/tasks/00_gather_custom_requirements.yml
@@ -20,12 +20,14 @@
     - not venv_migrate_show_diff_with_default
     - __venv_migrate_virtualenvs | length > 1
   register: __venv_migrate_freeze_results
+  changed_when: false  # only gathering information
 
 - name: Gather pip freeze output from all venvs including the default venv
   ansible.builtin.command: "{{ item }}/bin/pip freeze"
   loop: "{{ __venv_migrate_virtualenvs }}"
   when: venv_migrate_show_diff_with_default | bool
   register: __venv_migrate_freeze_results_default
+  changed_when: false  # only gathering information
 
 - name: Gather results from pip_freeze based on conditions
   ansible.builtin.set_fact: