add yamllint to project pipeline and fix errors

.github/ISSUE_TEMPLATE/bug_report.yaml

@@ -67,7 +67,7 @@ body:
         Describe in detail the exact steps that someone else can take to
         reproduce this bug using the current stable release of the Ansible NetBox collection.
         Include any sanatized playbooks, variables, & tasks specifically showing just the issue.
-      #placeholder: |
+      # placeholder: |
 
     validations:
       required: true

.github/workflows/main.yml

@@ -15,6 +15,13 @@ jobs:
       - name: Run black
         uses: psf/black@stable
 
+      - name: Run yamllint
+        uses: ibiqlik/action-yamllint@v3
+        with:
+          file_or_dir: .
+          config_file: .yamllint.yml
+          strict: true
+
   unit_testing:
     runs-on: ubuntu-24.04
     needs: linting
@@ -66,7 +73,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        include:                              
+        include:
           - VERSION: "v4.1"
             NETBOX_DOCKER_VERSION: 3.0.1
           - VERSION: "v4.2"
@@ -114,7 +121,7 @@ jobs:
           docker container ls
           docker logs netbox-docker-netbox-1
           timeout 300 bash -c 'while [[ "$(curl -s -o /dev/null -w ''%{http_code}'' localhost:32768/login/)" != "200" ]]; do echo "waiting for NetBox"; sleep 5; done' || false
-        working-directory: /home/runner/.ansible/collections/ansible_collections/netbox/netbox        
+        working-directory: /home/runner/.ansible/collections/ansible_collections/netbox/netbox
 
       - name: Pre-populate NetBox
         run: ./tests/integration/netbox-deploy.py
@@ -126,13 +133,13 @@ jobs:
           ansible-test integration -v --color --coverage --python ${{ env.python-version }} inventory-${{ matrix.VERSION }}
         working-directory: /home/runner/.ansible/collections/ansible_collections/netbox/netbox
 
-      - name: Run regression tests        
+      - name: Run regression tests
         continue-on-error: true
-        run: |          
-          ansible-test integration -v --color --coverage --python ${{ env.python-version }} regression-${{ matrix.VERSION }}          
+        run: |
+          ansible-test integration -v --color --coverage --python ${{ env.python-version }} regression-${{ matrix.VERSION }}
         working-directory: /home/runner/.ansible/collections/ansible_collections/netbox/netbox
 
-      - name: Run integration tests        
+      - name: Run integration tests
         run: |
           ansible-test integration -v --color --coverage --python ${{ env.python-version }} ${{ matrix.VERSION }}
           ansible-test coverage report --all --omit "tests/*,hacking/*,docs/*" --show-missing

.github/workflows/tests.yml

@@ -47,4 +47,4 @@ jobs:
           '${{ needs.unit-galaxy.result }}',
           '${{ needs.unit-source.result }}',
           '${{ needs.ansible-lint.result }}'
-          ])"
+          ])"

.yamllint.yml

@@ -0,0 +1,52 @@
+---
+extends: default
+
+# see rules at https://yamllint.readthedocs.io/en/latest/rules.html
+ignore: |
+  - .cache/
+  - .venv/
+  - .github/
+
+rules:
+  braces:
+    max-spaces-inside: 20
+    level: error
+  brackets:
+    max-spaces-inside: 20
+    level: error
+  colons:
+    max-spaces-before: 1
+    max-spaces-after: 20
+    level: error
+  commas:
+    max-spaces-before: 50
+    min-spaces-after: 1
+    max-spaces-after: 20
+    level: error
+  comments:
+    min-spaces-from-content: 1
+  comments-indentation: disable
+  document-start:
+    present: true
+  empty-lines:
+    max: 3
+    level: error
+  hyphens:
+    level: error
+  indentation:
+    spaces: 2
+    indent-sequences: whatever
+    check-multi-line-strings: false
+  key-duplicates: enable
+  line-length:
+    max: 230
+    level: warning
+    allow-non-breakable-words: true
+    allow-non-breakable-inline-mappings: true
+  new-line-at-end-of-file: enable
+  new-lines:
+    type: unix
+  trailing-spaces: enable
+  truthy: disable
+  octal-values:
+    forbid-explicit-octal: true

changelogs/changelog.yaml

@@ -1,3 +1,4 @@
+---
 ancestor: null
 releases:
   0.1.0:

changelogs/fragments/1182-fix-contact-groups.yml

@@ -1,3 +1,4 @@
+---
 minor_changes:
   - Change `netbox_contact.contact_group` to `contact_groups`
   - Add integration tests for contact groups

changelogs/fragments/1433-add-custom-headers.yml

@@ -1,2 +1,3 @@
+---
 minor_changes:
   - Add support for custom headers for all modules

changelogs/fragments/1435-fix-duplicate-task-name.yml

@@ -1,2 +1,3 @@
+---
 bugfixes:
   - Fix task duplicate task name in documentation that cause ansible-lint error

changelogs/fragments/1438-fix-inventory-service-collection.yml

@@ -1,2 +1,3 @@
+---
 bugfixes:
   - nb_inventory - Fix service collection for version greater than 4.3

changelogs/fragments/1439-fix-sevral-bug-discovered-by-integration-tests.yml

@@ -1,3 +1,4 @@
+---
 bugfixes:
   - Add netbox version check to support service creation for netbox version prior of 4.3
   - Use dedicated function to check netbox version istead of self.full_version for rack.

changelogs/fragments/1459-add-netbox-data-sources.yml

@@ -1,2 +1,3 @@
+---
 minor_changes:
   - netbox_data_source - New module `#1459 <https://github.com/netbox-community/ansible_modules/pull/1459>`

changelogs/fragments/1462-fix-pylint-errors.yml

@@ -1,2 +1,3 @@
+---
 minor_changes:
   - Fix ansible-bad-import-from pylint errors

changelogs/fragments/1464-fix-mac-lookup-by-id.yml

@@ -1,2 +1,3 @@
+---
 bugfixes:
   - "nb_device_interface: Fix specifying primary_mac_address objects by id for disambiguation"

changelogs/fragments/1469-fix-old-apis.yml

@@ -1,2 +1,3 @@
+---
 minor_changes:
   - Fix broken code path when using old api path on old netbox systems

changelogs/fragments/contacts.yml

@@ -1,2 +1,3 @@
+---
 minor_changes:
   - netbox_contact_assignment - New module `#1480 <https://github.com/netbox-community/ansible_modules/pull/1480>`

changelogs/fragments/nb_inventory.yml

@@ -1,2 +1,3 @@
+---
 bugfixes:
   - nb_inventory - Fixed empty inventory results when netbox server URL is a non-root path

changelogs/fragments/netbox_circuit_termination.yml

@@ -1,2 +1,3 @@
+---
 minor_changes:
   - netbox_circuit_termination - Add parameters termination_id and termination_type for NetBox 4.2+

changelogs/fragments/netbox_custom_field.yml

@@ -1,2 +1,3 @@
+---
 minor_changes:
   - Support for related_object_filter when related_object_type is "object"

changelogs/fragments/netbox_service.yml

@@ -1,2 +1,3 @@
+---
 bugfixes:
   - netbox_service - Fix issue 1426 - broken netbox_service module

changelogs/fragments/netbox_tag.yml

@@ -1,2 +1,3 @@
+---
 minor_changes:
   - netbox_tag - Add support for object_types on tags

changelogs/fragments/users-ordering.yml

@@ -1,2 +1,3 @@
+---
 minor_changes:
   - user.groups, user.permissions, user_group.permissions, permission.actions, and permission.object_types are now treated as unordered sets for update comparison purposes.

changelogs/fragments/yamllint.yml

@@ -0,0 +1,3 @@
+---
+minor_changes:
+  - add yamllint to project pipeline.

requirements.yml

@@ -1,2 +1,3 @@
+---
 collections:
   - name: .