Skip to content

ci: merge travis matrix, add salt-lint & rubocop to lint job#18

Merged
myii merged 5 commits intosaltstack-formulas:masterfrom
myii:ci/merge-matrix-and-add-salt-lint-and-rubocop
Oct 9, 2019
Merged

ci: merge travis matrix, add salt-lint & rubocop to lint job#18
myii merged 5 commits intosaltstack-formulas:masterfrom
myii:ci/merge-matrix-and-add-salt-lint-and-rubocop

Conversation

@myii
Copy link

@myii myii commented Oct 9, 2019

myii added 5 commits October 9, 2019 18:49
```bash
Examining ufw/config/applications.sls of type state
[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/applications.sls:30
ufw-app-{{method}}-{{app_name}}-{{from_addr}}:

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/applications.sls:32
ufw-app-{{method}}-{{app_name}}:

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/applications.sls:34
  ufw.{{method}}:

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/applications.sls:35
    - app: '"{{app_name}}"'

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/applications.sls:37
    - from_addr: {{from_addr}}

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/applications.sls:40
    - to_addr: {{to_addr}}

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/applications.sls:45
    - comment: '"{{comment}}"'
```
```bash
Examining ufw/config/interfaces.sls of type state
[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/interfaces.sls:20
ufw-interface-{{interface_name}}:

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/interfaces.sls:22
    - interface: {{interface_name}}

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/interfaces.sls:24
    - comment: '"{{comment}}"'
```
```bash
Examining ufw/config/open.sls of type state
[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/open.sls:20
ufw-open-{{open_addr}}:

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/open.sls:22
    - from_addr: {{open_addr}}

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/open.sls:24
    - comment: '"{{comment}}"'
```
```bash
Examining ufw/config/services.sls of type state
[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/services.sls:32
ufw-svc-{{method}}-{{service_name}}-{{from_addr}}:

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/services.sls:33
  ufw.{{method}}:

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/services.sls:35
    - protocol: {{protocol}}

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/services.sls:38
    - from_addr: {{from_addr}}

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/services.sls:41
    - from_port: "{{from_port}}"

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/services.sls:44
    - to_addr: {{to_addr}}

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/services.sls:49
    - comment: '"{{comment}}"'

[206] Jinja variables should have spaces before and after: {{ var_name }}
ufw/config/services.sls:51
    - to_port: "{{to_port}}"
```
@myii myii force-pushed the ci/merge-matrix-and-add-salt-lint-and-rubocop branch from 880f58d to 7b1b3c9 Compare October 9, 2019 22:25
@myii myii merged commit 51fc6de into saltstack-formulas:master Oct 9, 2019
@saltstack-formulas-travis

🎉 This PR is included in version 0.5.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants