Skip to content

install_nextcloud: all variable names should start with install_nextcloud_ #365

Open
@RussellAult

Description

@RussellAult

Description

Currently, many of the install_nextcloud role's internal variables are prefixed with nextcloud_, but ansible-lint's var-naming rule specifies that a prefix of role_name_ should be applied to all role-specific variables. This causes ansible-lint to throw errors if the role is includeed or imported with any vars parameters.

Steps to reproduce

# roles/custom_nextcloud/tasks/main.yml

- name: Install Nextcloud
  ansible.builtin.include_role:
    name: nextcloud.admin.install_nextcloud
  vars:
    nextcloud_disable_websrv_default_site: true

Expected outcome

No ansible-lint errors

Actual outcome

$ roles/custom_nextcloud/tasks/main.yml
WARNING  Listing 1 violation(s) that are fatal
var-naming[no-role-prefix]: Variables names from within roles should use role_name_ as a prefix. (vars: nextcloud_disable_websrv_default_site)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions