Skip to content
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.

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

auto cap with name[prefix] rule bug #3921

Closed
djdanielsson opened this issue Nov 29, 2023 · 0 comments · Fixed by #3922
Closed

auto cap with name[prefix] rule bug #3921

djdanielsson opened this issue Nov 29, 2023 · 0 comments · Fixed by #3922
Assignees
Labels

Comments

@djdanielsson
Copy link

Summary

If you have the name[prefix] rule enabled and run the fix it will auto capitalize the prefix if the next character after prefix is not capitalized.

Issue Type
  • Bug Report
OS / ENVIRONMENT
ansible-lint --version 6.22.0 using Ansible-core:2.15.6 ansible-compat:4.1.10 ruamel-yaml:0.18.5 ruamel-yaml-clib:0.2.8
  • ansible installation method: pip
  • ansible-lint installation method: pip
STEPS TO REPRODUCE

test.yml

- name: test | not cap
  ansible.builtin.debug:
    msg: "not cap"

- name: test | Cap
  ansible.builtin.debug:
    msg: "Cap"

will turn into

- name: Test | not cap
  ansible.builtin.debug:
    msg: "not cap"

- name: test | Cap
  ansible.builtin.debug:
    msg: "Cap"
Desired Behavior

test.yml

- name: test | Not cap
  ansible.builtin.debug:
    msg: "not cap"

- name: test | Cap
  ansible.builtin.debug:
    msg: "Cap"
Actual Behavior
- name: Test | not cap
  ansible.builtin.debug:
    msg: "not cap"

- name: test | Cap
  ansible.builtin.debug:
    msg: "Cap"
@djdanielsson djdanielsson added bug new Triage required labels Nov 29, 2023
@ssbarnea ssbarnea assigned ssbarnea and audgirka and unassigned ssbarnea Nov 30, 2023
@ssbarnea ssbarnea removed the new Triage required label Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants