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

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

Conversation

myii
Copy link
Member

@myii myii commented Oct 9, 2019

@myii myii force-pushed the ci/merge-matrix-and-add-salt-lint-and-rubocop branch 5 times, most recently from 587e434 to 284ac78 Compare October 9, 2019 21:18
myii added 4 commits October 9, 2019 22:27
```bash
Examining users/googleauth.sls of type state
[201] Trailing whitespace
users/googleauth.sls:9
      - file: {{ users.googleauth_dir }}

[204] Lines should be no longer that 160 chars
users/googleauth.sls:26
    - repl: "auth       [success=done new_authtok_reqd=done default=die]   pam_google_authenticator.so user=root secret={{ users.googleauth_dir }}/${USER}_{{ svc }} echo_verification_code\n@include common-auth"
```
```bash
Examining users/init.sls of type state
[206] Jinja variables should have spaces before and after: {{ var_name }}
users/init.sls:176
    - expire: {{ (user['expire'] / 86400) | int}}

[206] Jinja variables should have spaces before and after: {{ var_name }}
users/init.sls:202
      - {{optional_group}}
```
```bash
Examining users/sudo.sls of type state
[201] Trailing whitespace
users/sudo.sls:13
      - file: {{ users.sudoers_dir }}

[201] Trailing whitespace
users/sudo.sls:22
        - name: {{ users.sudoers_file }}
```
@myii myii force-pushed the ci/merge-matrix-and-add-salt-lint-and-rubocop branch from 284ac78 to f17d156 Compare October 9, 2019 21:28
@myii myii merged commit 5e6846b into saltstack-formulas:master Oct 9, 2019
@saltstack-formulas-travis

🎉 This PR is included in version 0.48.1 🎉

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