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

WIP: Re-add rule 215 #217

Closed
wants to merge 7 commits into from
Closed

WIP: Re-add rule 215 #217

wants to merge 7 commits into from

Conversation

roaldnefs
Copy link
Member

@roaldnefs roaldnefs commented Dec 11, 2020

Fix rule 215 (using 'replace: False' is required when not specifying content):

  • Allow less indented YAML.

Note: this pull request is still work in progress!

roaldnefs and others added 7 commits December 11, 2020 14:54
Signed-off-by: Roald Nefs <info@roaldnefs.com>
Signed-off-by: Roald Nefs <info@roaldnefs.com>
Signed-off-by: Roald Nefs <info@roaldnefs.com>
Update supported Python version to include newer Python versions.

This commit also fixes some code style issues found by flake 8.

Signed-off-by: Roald Nefs <info@roaldnefs.com>
roaldnefs added a commit to roaldnefs/salt-lint that referenced this pull request Jan 3, 2021
Remove rule 215 as it currently causes false positives when states are
mixed with Jinja. The rule will be re-added in warpnet#217 once it is fixed.

Signed-off-by: Roald Nefs <info@roaldnefs.com>
@roaldnefs roaldnefs changed the title WIP: Fix rule 215 WIP: Re-add rule 215 Jan 16, 2021
@roaldnefs roaldnefs changed the base branch from develop to main January 17, 2021 09:01
@roaldnefs roaldnefs added Priority: Low Status: Invalid This doesn't seem right Type: Enhancement New feature or request labels Jan 29, 2021
@roaldnefs roaldnefs closed this Dec 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Low Status: Invalid This doesn't seem right Type: Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant