From 4e48a7adbbfc25c4c968d7a48bf6ecc0b4edd9fa Mon Sep 17 00:00:00 2001 From: Imran Iqbal Date: Wed, 9 Oct 2019 15:45:51 +0100 Subject: [PATCH 1/5] fix(composer.sls): fix `salt-lint` errors ```bash Examining php/composer.sls of type state [210] Numbers that start with `0` should always be encapsulated in quotation marks php/composer.sls:27 - mode: 0755 ``` --- php/composer.sls | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/php/composer.sls b/php/composer.sls index 2b61c6f..9640537 100644 --- a/php/composer.sls +++ b/php/composer.sls @@ -24,7 +24,7 @@ include: get-composer: file.managed: - name: {{ php.lookup.pkgs.temp_dir }}/installer - - mode: 0755 + - mode: '0755' - unless: test -f {{ install_file }} - source: https://getcomposer.org/installer - source_hash: https://composer.github.io/installer.sig From b4d994cc09c0ef11b13361f775346e9cb0035e3a Mon Sep 17 00:00:00 2001 From: Imran Iqbal Date: Wed, 9 Oct 2019 15:52:10 +0100 Subject: [PATCH 2/5] fix(repo.sls): fix `salt-lint` errors ```bash Examining php/hhvm/repo.sls of type state [206] Jinja variables should have spaces before and after: {{ var_name }} php/hhvm/repo.sls:12 - name: deb http://dl.hhvm.com/{{ salt['grains.get']('os')|lower }} {{ salt['grains.get']('oscodename')}} main ``` --- php/hhvm/repo.sls | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/php/hhvm/repo.sls b/php/hhvm/repo.sls index 84c2b61..bea7f7f 100644 --- a/php/hhvm/repo.sls +++ b/php/hhvm/repo.sls @@ -9,7 +9,7 @@ include: hhvm_repo: pkgrepo.managed: - - name: deb http://dl.hhvm.com/{{ salt['grains.get']('os')|lower }} {{ salt['grains.get']('oscodename')}} main + - name: deb http://dl.hhvm.com/{{ salt['grains.get']('os')|lower }} {{ salt['grains.get']('oscodename') }} main - file: /etc/apt/sources.list.d/hhvm.list - keyid: 0x5a16e7281be7a449 - keyserver: keyserver.ubuntu.com From 01f5ede277a607d1a326c43538ab1f8860f99d0b Mon Sep 17 00:00:00 2001 From: Imran Iqbal Date: Wed, 9 Oct 2019 16:00:28 +0100 Subject: [PATCH 3/5] fix(map.jinja): fix `salt-lint` errors ```bash Examining php/map.jinja of type state [204] Lines should be no longer that 160 chars php/map.jinja:25 'build_pkgs': ['libssl-dev', 'libcurl4-openssl-dev', 'pkg-config', 'libsslcommon2-dev', 'gcc', 'make', 'autoconf', 'libc-dev', 'pkg-config'], [204] Lines should be no longer that 160 chars php/map.jinja:191 'disable_functions': ['pcntl_alarm','pcntl_fork','pcntl_waitpid','pcntl_wait','pcntl_wifexited','pcntl_wifstopped','pcntl_wifsignaled', [204] Lines should be no longer that 160 chars php/map.jinja:192 'pcntl_wexitstatus','pcntl_wtermsig','pcntl_wstopsig','pcntl_signal','pcntl_signal_dispatch','pcntl_get_last_error','pcntl_strerror', [204] Lines should be no longer that 160 chars php/map.jinja:520 'disable_functions': ['pcntl_alarm','pcntl_fork','pcntl_waitpid','pcntl_wait','pcntl_wifexited','pcntl_wifstopped','pcntl_wifsignaled', [204] Lines should be no longer that 160 chars php/map.jinja:521 'pcntl_wexitstatus','pcntl_wtermsig','pcntl_wstopsig','pcntl_signal','pcntl_signal_dispatch','pcntl_get_last_error','pcntl_strerror', [204] Lines should be no longer that 160 chars php/map.jinja:896 'disable_functions': ['pcntl_alarm','pcntl_fork','pcntl_waitpid','pcntl_wait','pcntl_wifexited','pcntl_wifstopped','pcntl_wifsignaled', [204] Lines should be no longer that 160 chars php/map.jinja:897 'pcntl_wexitstatus','pcntl_wtermsig','pcntl_wstopsig','pcntl_signal','pcntl_signal_dispatch','pcntl_get_last_error','pcntl_strerror', [204] Lines should be no longer that 160 chars php/map.jinja:1270 'disable_functions': ['pcntl_alarm','pcntl_fork','pcntl_waitpid','pcntl_wait','pcntl_wifexited','pcntl_wifstopped','pcntl_wifsignaled', [204] Lines should be no longer that 160 chars php/map.jinja:1271 'pcntl_wexitstatus','pcntl_wtermsig','pcntl_wstopsig','pcntl_signal','pcntl_signal_dispatch','pcntl_get_last_error','pcntl_strerror', [204] Lines should be no longer that 160 chars php/map.jinja:1644 'disable_functions': ['pcntl_alarm','pcntl_fork','pcntl_waitpid','pcntl_wait','pcntl_wifexited','pcntl_wifstopped','pcntl_wifsignaled', [204] Lines should be no longer that 160 chars php/map.jinja:1645 'pcntl_wexitstatus','pcntl_wtermsig','pcntl_wstopsig','pcntl_signal','pcntl_signal_dispatch','pcntl_get_last_error','pcntl_strerror', [204] Lines should be no longer that 160 chars php/map.jinja:2035 'disable_functions': ['pcntl_alarm','pcntl_fork','pcntl_waitpid','pcntl_wait','pcntl_wifexited','pcntl_wifstopped','pcntl_wifsignaled', [204] Lines should be no longer that 160 chars php/map.jinja:2036 'pcntl_wexitstatus','pcntl_wtermsig','pcntl_wstopsig','pcntl_signal','pcntl_signal_dispatch','pcntl_get_last_error','pcntl_strerror', [204] Lines should be no longer that 160 chars php/map.jinja:2206 'build_pkgs': ['libssl-dev', 'libcurl4-openssl-dev', 'pkg-config', 'libsslcommon2-dev', 'gcc', 'make', 'autoconf', 'libc-dev', 'pkg-config'], [204] Lines should be no longer that 160 chars php/map.jinja:2375 'disable_functions': ['pcntl_alarm','pcntl_fork','pcntl_waitpid','pcntl_wait','pcntl_wifexited','pcntl_wifstopped','pcntl_wifsignaled', [204] Lines should be no longer that 160 chars php/map.jinja:2376 'pcntl_wexitstatus','pcntl_wtermsig','pcntl_wstopsig','pcntl_signal','pcntl_signal_dispatch','pcntl_get_last_error','pcntl_strerror', ``` --- php/map.jinja | 199 ++++++++++++++++++++++++++++++++++++++++++++------ 1 file changed, 176 insertions(+), 23 deletions(-) diff --git a/php/map.jinja b/php/map.jinja index 0b4cb18..fa5a068 100644 --- a/php/map.jinja +++ b/php/map.jinja @@ -22,7 +22,17 @@ 'apc': 'php-apcu', 'apcu': 'php-apcu-bc', 'bcmath': 'php' + php_version + '-bcmath', - 'build_pkgs': ['libssl-dev', 'libcurl4-openssl-dev', 'pkg-config', 'libsslcommon2-dev', 'gcc', 'make', 'autoconf', 'libc-dev', 'pkg-config'], + 'build_pkgs': [ + 'libssl-dev', + 'libcurl4-openssl-dev', + 'pkg-config', + 'libsslcommon2-dev', + 'gcc', + 'make', + 'autoconf', + 'libc-dev', + 'pkg-config', + ], 'bz2': 'php' + php_version + '-bz2', 'cache-lite': 'php-cache-lite', 'cgi': 'php' + php_version + '-cgi', @@ -188,9 +198,28 @@ 'auto_globals_jit': 'On', 'default_mimetype': '"text/html"', 'default_socket_timeout': 60, - 'disable_functions': ['pcntl_alarm','pcntl_fork','pcntl_waitpid','pcntl_wait','pcntl_wifexited','pcntl_wifstopped','pcntl_wifsignaled', - 'pcntl_wexitstatus','pcntl_wtermsig','pcntl_wstopsig','pcntl_signal','pcntl_signal_dispatch','pcntl_get_last_error','pcntl_strerror', - 'pcntl_sigprocmask','pcntl_sigwaitinfo','pcntl_sigtimedwait','pcntl_exec','pcntl_getpriority','pcntl_setpriority'], + 'disable_functions': [ + 'pcntl_alarm', + 'pcntl_fork', + 'pcntl_waitpid', + 'pcntl_wait', + 'pcntl_wifexited', + 'pcntl_wifstopped', + 'pcntl_wifsignaled', + 'pcntl_wexitstatus', + 'pcntl_wtermsig', + 'pcntl_wstopsig', + 'pcntl_signal', + 'pcntl_signal_dispatch', + 'pcntl_get_last_error', + 'pcntl_strerror', + 'pcntl_sigprocmask', + 'pcntl_sigwaitinfo', + 'pcntl_sigtimedwait', + 'pcntl_exec', + 'pcntl_getpriority', + 'pcntl_setpriority', + ], 'display_errors': 'Off', 'display_startup_errors': 'Off', 'enable_dl': 'Off', @@ -517,9 +546,28 @@ 'auto_globals_jit': 'On', 'default_mimetype': '"text/html"', 'default_socket_timeout': 60, - 'disable_functions': ['pcntl_alarm','pcntl_fork','pcntl_waitpid','pcntl_wait','pcntl_wifexited','pcntl_wifstopped','pcntl_wifsignaled', - 'pcntl_wexitstatus','pcntl_wtermsig','pcntl_wstopsig','pcntl_signal','pcntl_signal_dispatch','pcntl_get_last_error','pcntl_strerror', - 'pcntl_sigprocmask','pcntl_sigwaitinfo','pcntl_sigtimedwait','pcntl_exec','pcntl_getpriority','pcntl_setpriority'], + 'disable_functions': [ + 'pcntl_alarm', + 'pcntl_fork', + 'pcntl_waitpid', + 'pcntl_wait', + 'pcntl_wifexited', + 'pcntl_wifstopped', + 'pcntl_wifsignaled', + 'pcntl_wexitstatus', + 'pcntl_wtermsig', + 'pcntl_wstopsig', + 'pcntl_signal', + 'pcntl_signal_dispatch', + 'pcntl_get_last_error', + 'pcntl_strerror', + 'pcntl_sigprocmask', + 'pcntl_sigwaitinfo', + 'pcntl_sigtimedwait', + 'pcntl_exec', + 'pcntl_getpriority', + 'pcntl_setpriority', + ], 'display_errors': 'Off', 'display_startup_errors': 'Off', 'enable_dl': 'Off', @@ -893,9 +941,28 @@ 'auto_globals_jit': 'On', 'default_mimetype': '"text/html"', 'default_socket_timeout': 60, - 'disable_functions': ['pcntl_alarm','pcntl_fork','pcntl_waitpid','pcntl_wait','pcntl_wifexited','pcntl_wifstopped','pcntl_wifsignaled', - 'pcntl_wexitstatus','pcntl_wtermsig','pcntl_wstopsig','pcntl_signal','pcntl_signal_dispatch','pcntl_get_last_error','pcntl_strerror', - 'pcntl_sigprocmask','pcntl_sigwaitinfo','pcntl_sigtimedwait','pcntl_exec','pcntl_getpriority','pcntl_setpriority'], + 'disable_functions': [ + 'pcntl_alarm', + 'pcntl_fork', + 'pcntl_waitpid', + 'pcntl_wait', + 'pcntl_wifexited', + 'pcntl_wifstopped', + 'pcntl_wifsignaled', + 'pcntl_wexitstatus', + 'pcntl_wtermsig', + 'pcntl_wstopsig', + 'pcntl_signal', + 'pcntl_signal_dispatch', + 'pcntl_get_last_error', + 'pcntl_strerror', + 'pcntl_sigprocmask', + 'pcntl_sigwaitinfo', + 'pcntl_sigtimedwait', + 'pcntl_exec', + 'pcntl_getpriority', + 'pcntl_setpriority', + ], 'display_errors': 'Off', 'display_startup_errors': 'Off', 'enable_dl': 'Off', @@ -1267,9 +1334,28 @@ 'auto_globals_jit': 'On', 'default_mimetype': '"text/html"', 'default_socket_timeout': 60, - 'disable_functions': ['pcntl_alarm','pcntl_fork','pcntl_waitpid','pcntl_wait','pcntl_wifexited','pcntl_wifstopped','pcntl_wifsignaled', - 'pcntl_wexitstatus','pcntl_wtermsig','pcntl_wstopsig','pcntl_signal','pcntl_signal_dispatch','pcntl_get_last_error','pcntl_strerror', - 'pcntl_sigprocmask','pcntl_sigwaitinfo','pcntl_sigtimedwait','pcntl_exec','pcntl_getpriority','pcntl_setpriority'], + 'disable_functions': [ + 'pcntl_alarm', + 'pcntl_fork', + 'pcntl_waitpid', + 'pcntl_wait', + 'pcntl_wifexited', + 'pcntl_wifstopped', + 'pcntl_wifsignaled', + 'pcntl_wexitstatus', + 'pcntl_wtermsig', + 'pcntl_wstopsig', + 'pcntl_signal', + 'pcntl_signal_dispatch', + 'pcntl_get_last_error', + 'pcntl_strerror', + 'pcntl_sigprocmask', + 'pcntl_sigwaitinfo', + 'pcntl_sigtimedwait', + 'pcntl_exec', + 'pcntl_getpriority', + 'pcntl_setpriority', + ], 'display_errors': 'Off', 'display_startup_errors': 'Off', 'enable_dl': 'Off', @@ -1641,9 +1727,28 @@ 'auto_globals_jit': 'On', 'default_mimetype': '"text/html"', 'default_socket_timeout': 60, - 'disable_functions': ['pcntl_alarm','pcntl_fork','pcntl_waitpid','pcntl_wait','pcntl_wifexited','pcntl_wifstopped','pcntl_wifsignaled', - 'pcntl_wexitstatus','pcntl_wtermsig','pcntl_wstopsig','pcntl_signal','pcntl_signal_dispatch','pcntl_get_last_error','pcntl_strerror', - 'pcntl_sigprocmask','pcntl_sigwaitinfo','pcntl_sigtimedwait','pcntl_exec','pcntl_getpriority','pcntl_setpriority'], + 'disable_functions': [ + 'pcntl_alarm', + 'pcntl_fork', + 'pcntl_waitpid', + 'pcntl_wait', + 'pcntl_wifexited', + 'pcntl_wifstopped', + 'pcntl_wifsignaled', + 'pcntl_wexitstatus', + 'pcntl_wtermsig', + 'pcntl_wstopsig', + 'pcntl_signal', + 'pcntl_signal_dispatch', + 'pcntl_get_last_error', + 'pcntl_strerror', + 'pcntl_sigprocmask', + 'pcntl_sigwaitinfo', + 'pcntl_sigtimedwait', + 'pcntl_exec', + 'pcntl_getpriority', + 'pcntl_setpriority', + ], 'display_errors': 'Off', 'display_startup_errors': 'Off', 'enable_dl': 'Off', @@ -2032,9 +2137,28 @@ 'auto_globals_jit': 'On', 'default_mimetype': '"text/html"', 'default_socket_timeout': 60, - 'disable_functions': ['pcntl_alarm','pcntl_fork','pcntl_waitpid','pcntl_wait','pcntl_wifexited','pcntl_wifstopped','pcntl_wifsignaled', - 'pcntl_wexitstatus','pcntl_wtermsig','pcntl_wstopsig','pcntl_signal','pcntl_signal_dispatch','pcntl_get_last_error','pcntl_strerror', - 'pcntl_sigprocmask','pcntl_sigwaitinfo','pcntl_sigtimedwait','pcntl_exec','pcntl_getpriority','pcntl_setpriority'], + 'disable_functions': [ + 'pcntl_alarm', + 'pcntl_fork', + 'pcntl_waitpid', + 'pcntl_wait', + 'pcntl_wifexited', + 'pcntl_wifstopped', + 'pcntl_wifsignaled', + 'pcntl_wexitstatus', + 'pcntl_wtermsig', + 'pcntl_wstopsig', + 'pcntl_signal', + 'pcntl_signal_dispatch', + 'pcntl_get_last_error', + 'pcntl_strerror', + 'pcntl_sigprocmask', + 'pcntl_sigwaitinfo', + 'pcntl_sigtimedwait', + 'pcntl_exec', + 'pcntl_getpriority', + 'pcntl_setpriority', + ], 'display_errors': 'Off', 'display_startup_errors': 'Off', 'enable_dl': 'Off', @@ -2203,7 +2327,17 @@ 'apcu': 'php-apcu-bc', 'bcmath': 'php' + php_version + '-bcmath', 'bz2': 'php' + php_version + '-bz2', - 'build_pkgs': ['libssl-dev', 'libcurl4-openssl-dev', 'pkg-config', 'libsslcommon2-dev', 'gcc', 'make', 'autoconf', 'libc-dev', 'pkg-config'], + 'build_pkgs': [ + 'libssl-dev', + 'libcurl4-openssl-dev', + 'pkg-config', + 'libsslcommon2-dev', + 'gcc', + 'make', + 'autoconf', + 'libc-dev', + 'pkg-config', + ], 'cache-lite': 'php-cache-lite', 'cgi': 'php' + php_version + '-cgi', 'cli': 'php' + php_version + '-cli', @@ -2372,9 +2506,28 @@ 'auto_globals_jit': 'On', 'default_mimetype': '"text/html"', 'default_socket_timeout': 60, - 'disable_functions': ['pcntl_alarm','pcntl_fork','pcntl_waitpid','pcntl_wait','pcntl_wifexited','pcntl_wifstopped','pcntl_wifsignaled', - 'pcntl_wexitstatus','pcntl_wtermsig','pcntl_wstopsig','pcntl_signal','pcntl_signal_dispatch','pcntl_get_last_error','pcntl_strerror', - 'pcntl_sigprocmask','pcntl_sigwaitinfo','pcntl_sigtimedwait','pcntl_exec','pcntl_getpriority','pcntl_setpriority'], + 'disable_functions': [ + 'pcntl_alarm', + 'pcntl_fork', + 'pcntl_waitpid', + 'pcntl_wait', + 'pcntl_wifexited', + 'pcntl_wifstopped', + 'pcntl_wifsignaled', + 'pcntl_wexitstatus', + 'pcntl_wtermsig', + 'pcntl_wstopsig', + 'pcntl_signal', + 'pcntl_signal_dispatch', + 'pcntl_get_last_error', + 'pcntl_strerror', + 'pcntl_sigprocmask', + 'pcntl_sigwaitinfo', + 'pcntl_sigtimedwait', + 'pcntl_exec', + 'pcntl_getpriority', + 'pcntl_setpriority', + ], 'display_errors': 'Off', 'display_startup_errors': 'Off', 'enable_dl': 'Off', From a5a8a959d64a5bd11bff7f7d785875da53308680 Mon Sep 17 00:00:00 2001 From: Imran Iqbal Date: Wed, 9 Oct 2019 16:09:06 +0100 Subject: [PATCH 4/5] ci: merge travis matrix, add `salt-lint` & `rubocop` to `lint` job * Semi-automated using https://github.com/myii/ssf-formula/pull/60 --- .rubocop.yml | 10 +++++ .salt-lint | 15 +++++++ .travis.yml | 109 +++++++++++++++++++++++++++++++-------------------- .yamllint | 1 + Gemfile | 7 ++-- bin/kitchen | 21 +++++----- 6 files changed, 109 insertions(+), 54 deletions(-) create mode 100644 .rubocop.yml create mode 100644 .salt-lint diff --git a/.rubocop.yml b/.rubocop.yml new file mode 100644 index 0000000..bdae9aa --- /dev/null +++ b/.rubocop.yml @@ -0,0 +1,10 @@ +# -*- coding: utf-8 -*- +# vim: ft=yaml +--- +# General overrides used across formulas in the org +Metrics/LineLength: + # Increase from default of `80` + # Based on https://github.com/PyCQA/flake8-bugbear#opinionated-warnings (`B950`) + Max: 88 + +# Any offenses that should be fixed, e.g. collected via. `rubocop --auto-gen-config` diff --git a/.salt-lint b/.salt-lint new file mode 100644 index 0000000..5bfd547 --- /dev/null +++ b/.salt-lint @@ -0,0 +1,15 @@ +# -*- coding: utf-8 -*- +# vim: ft=yaml +--- +exclude_paths: + # Save time spent checking files by avoiding the deprecated `ng/` directory + - php/ng/ +skip_list: + # Using `salt-lint` for linting other files as well, such as Jinja macros/templates + - 205 # Use ".sls" as a Salt State file extension + # Skipping `207` and `208` because `210` is sufficient, at least for the time-being + # I.e. Allows 3-digit unquoted codes to still be used, such as `644` and `755` + - 207 # File modes should always be encapsulated in quotation marks + - 208 # File modes should always contain a leading zero +tags: [] +verbosity: 1 diff --git a/.travis.yml b/.travis.yml index 19db874..7c5c249 100644 --- a/.travis.yml +++ b/.travis.yml @@ -1,60 +1,43 @@ # -*- coding: utf-8 -*- # vim: ft=yaml --- +## Machine config dist: bionic -stages: - - test - - lint - - name: release - if: branch = master AND type != pull_request - sudo: required -cache: bundler -language: ruby - services: - docker -# Make sure the instances listed below match up with -# the `platforms` defined in `kitchen.yml` -env: - matrix: - - INSTANCE: debian-debian-10-develop-py3 - # - INSTANCE: ubuntu-ubuntu-1804-develop-py3 - # - INSTANCE: redhat-centos-7-develop-py3 - # - INSTANCE: redhat-fedora-30-develop-py3 - # - INSTANCE: suse-opensuse-leap-15-develop-py3 - # - INSTANCE: redhat-amazonlinux-2-develop-py2 - # - INSTANCE: debian-debian-9-2019-2-py3 - - INSTANCE: ubuntu-ubuntu-1804-2019-2-py3 - # - INSTANCE: redhat-centos-7-2019-2-py3 - # - INSTANCE: redhat-fedora-30-2019-2-py3 - # - INSTANCE: suse-opensuse-leap-15-2019-2-py3 - - INSTANCE: redhat-amazonlinux-2-2019-2-py2 - # - INSTANCE: debian-debian-9-2018-3-py2 - # - INSTANCE: ubuntu-ubuntu-1604-2018-3-py2 - # - INSTANCE: redhat-centos-7-2018-3-py2 - - INSTANCE: redhat-fedora-29-2018-3-py2 - - INSTANCE: suse-opensuse-leap-15-2018-3-py2 - # - INSTANCE: redhat-amazonlinux-2-2018-3-py2 - # - INSTANCE: debian-debian-8-2017-7-py2 - # - INSTANCE: ubuntu-ubuntu-1604-2017-7-py2 - - INSTANCE: redhat-centos-6-2017-7-py2 - # - INSTANCE: redhat-fedora-29-2017-7-py2 - # - INSTANCE: suse-opensuse-leap-15-2017-7-py2 - # - INSTANCE: redhat-amazonlinux-2-2017-7-py2 +## Language and cache config +language: ruby +cache: bundler +## Script to run for the test stage script: - - bin/kitchen verify ${INSTANCE} + - bin/kitchen verify "${INSTANCE}" +## Stages and jobs matrix +stages: + - test + - name: release + if: branch = master AND type != pull_request jobs: + allow_failures: + - env: Lint_rubocop + fast_finish: true include: - # Define the `lint` stage (runs `yamllint` and `commitlint`) - - stage: lint - language: node_js + ## Define the test stage that runs the linters (and testing matrix, if applicable) + + # Run all of the linters in a single job (except `rubocop`) + - language: node_js node_js: lts/* + env: Lint + name: 'Lint: salt-lint, yamllint & commitlint' before_install: skip script: + # Install and run `salt-lint` + - pip install --user salt-lint + - git ls-files | grep '\.sls$\|\.jinja$\|\.j2$\|\.tmpl$' + | xargs -I {} salt-lint {} # Install and run `yamllint` # Need at least `v1.17.0` for the `yaml-files` setting - pip install --user yamllint>=1.17.0 @@ -63,10 +46,52 @@ jobs: - npm install @commitlint/config-conventional -D - npm install @commitlint/travis-cli -D - commitlint-travis - # Define the release stage that runs `semantic-release` + # Run the `rubocop` linter in a separate job that is allowed to fail + # Once these lint errors are fixed, this can be merged into a single job + - language: node_js + node_js: lts/* + env: Lint_rubocop + name: 'Lint: rubocop' + before_install: skip + script: + # Install and run `rubocop` + - gem install rubocop + - rubocop -d + + ## Define the rest of the matrix based on Kitchen testing + # Make sure the instances listed below match up with + # the `platforms` defined in `kitchen.yml` + - env: INSTANCE=debian-debian-10-develop-py3 + # - env: INSTANCE=ubuntu-ubuntu-1804-develop-py3 + # - env: INSTANCE=redhat-centos-7-develop-py3 + # - env: INSTANCE=redhat-fedora-30-develop-py3 + # - env: INSTANCE=suse-opensuse-leap-15-develop-py3 + # - env: INSTANCE=redhat-amazonlinux-2-develop-py2 + # - env: INSTANCE=debian-debian-9-2019-2-py3 + - env: INSTANCE=ubuntu-ubuntu-1804-2019-2-py3 + # - env: INSTANCE=redhat-centos-7-2019-2-py3 + # - env: INSTANCE=redhat-fedora-30-2019-2-py3 + # - env: INSTANCE=suse-opensuse-leap-15-2019-2-py3 + - env: INSTANCE=redhat-amazonlinux-2-2019-2-py2 + # - env: INSTANCE=debian-debian-9-2018-3-py2 + # - env: INSTANCE=ubuntu-ubuntu-1604-2018-3-py2 + # - env: INSTANCE=redhat-centos-7-2018-3-py2 + - env: INSTANCE=redhat-fedora-29-2018-3-py2 + - env: INSTANCE=suse-opensuse-leap-15-2018-3-py2 + # - env: INSTANCE=redhat-amazonlinux-2-2018-3-py2 + # - env: INSTANCE=debian-debian-8-2017-7-py2 + # - env: INSTANCE=ubuntu-ubuntu-1604-2017-7-py2 + - env: INSTANCE=redhat-centos-6-2017-7-py2 + # - env: INSTANCE=redhat-fedora-29-2017-7-py2 + # - env: INSTANCE=suse-opensuse-leap-15-2017-7-py2 + # - env: INSTANCE=redhat-amazonlinux-2-2017-7-py2 + + ## Define the release stage that runs `semantic-release` - stage: release language: node_js node_js: lts/* + env: Release + name: 'Run semantic-release inc. file updates to AUTHORS, CHANGELOG & FORMULA' before_install: skip script: # Update `AUTHORS.md` diff --git a/.yamllint b/.yamllint index c16f39b..740beca 100644 --- a/.yamllint +++ b/.yamllint @@ -17,6 +17,7 @@ yaml-files: # Default settings - '*.yaml' - '*.yml' + - .salt-lint - .yamllint # SaltStack Formulas additional settings - '*.example' diff --git a/Gemfile b/Gemfile index 3b36de3..5a232b6 100644 --- a/Gemfile +++ b/Gemfile @@ -1,6 +1,7 @@ -source "https://rubygems.org" +# frozen_string_literal: true + +source 'https://rubygems.org' gem 'kitchen-docker', '>= 2.9' -gem 'kitchen-salt', '>= 0.6.0' gem 'kitchen-inspec', '>= 1.1' - +gem 'kitchen-salt', '>= 0.6.0' diff --git a/bin/kitchen b/bin/kitchen index 1cd44f3..dcfdb4c 100755 --- a/bin/kitchen +++ b/bin/kitchen @@ -8,22 +8,25 @@ # this file is here to facilitate running it. # -require "pathname" -ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile", - Pathname.new(__FILE__).realpath) +require 'pathname' +ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile', + Pathname.new(__FILE__).realpath) -bundle_binstub = File.expand_path("../bundle", __FILE__) +bundle_binstub = File.expand_path('bundle', __dir__) if File.file?(bundle_binstub) if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/ load(bundle_binstub) else - abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run. -Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this command again.") + abort( + 'Your `bin/bundle` was not generated by Bundler, '\ + 'so this binstub cannot run. Replace `bin/bundle` by running '\ + '`bundle binstubs bundler --force`, then run this command again.' + ) end end -require "rubygems" -require "bundler/setup" +require 'rubygems' +require 'bundler/setup' -load Gem.bin_path("test-kitchen", "kitchen") +load Gem.bin_path('test-kitchen', 'kitchen') From d78e96fa0c01823f1385b29904276aaeff9b77ad Mon Sep 17 00:00:00 2001 From: semantic-release-bot Date: Thu, 10 Oct 2019 03:15:10 +0000 Subject: [PATCH 5/5] chore(release): 1.2.2 [skip ci] ## [1.2.2](https://github.com/saltstack-formulas/php-formula/compare/v1.2.1...v1.2.2) (2019-10-10) ### Bug Fixes * **composer.sls:** fix `salt-lint` errors ([](https://github.com/saltstack-formulas/php-formula/commit/4e48a7a)) * **map.jinja:** fix `salt-lint` errors ([](https://github.com/saltstack-formulas/php-formula/commit/01f5ede)) * **repo.sls:** fix `salt-lint` errors ([](https://github.com/saltstack-formulas/php-formula/commit/b4d994c)) ### Continuous Integration * merge travis matrix, add `salt-lint` & `rubocop` to `lint` job ([](https://github.com/saltstack-formulas/php-formula/commit/a5a8a95)) --- AUTHORS.md | 4 ++-- CHANGELOG.md | 14 ++++++++++++++ FORMULA | 2 +- docs/AUTHORS.rst | 4 ++-- docs/CHANGELOG.rst | 17 +++++++++++++++++ 5 files changed, 36 insertions(+), 5 deletions(-) diff --git a/AUTHORS.md b/AUTHORS.md index 947dfc8..f93ccd7 100644 --- a/AUTHORS.md +++ b/AUTHORS.md @@ -6,7 +6,7 @@ Avatar|Contributor|Contributions :-:|---|:-: @gravyboat|[@gravyboat](https://github.com/gravyboat)|52 @aboe76|[@aboe76](https://github.com/aboe76)|48 -@myii|[@myii](https://github.com/myii)|36 +@myii|[@myii](https://github.com/myii)|44 @alxwr|[@alxwr](https://github.com/alxwr)|31 @n-rodriguez|[@n-rodriguez](https://github.com/n-rodriguez)|16 @nmadhok|[@nmadhok](https://github.com/nmadhok)|14 @@ -64,4 +64,4 @@ Avatar|Contributor|Contributions --- -Auto-generated by a [forked version](https://github.com/myii/maintainer) of [gaocegege/maintainer](https://github.com/gaocegege/maintainer) on 2019-10-07. +Auto-generated by a [forked version](https://github.com/myii/maintainer) of [gaocegege/maintainer](https://github.com/gaocegege/maintainer) on 2019-10-10. diff --git a/CHANGELOG.md b/CHANGELOG.md index 2ee3da5..133ec16 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,5 +1,19 @@ # Changelog +## [1.2.2](https://github.com/saltstack-formulas/php-formula/compare/v1.2.1...v1.2.2) (2019-10-10) + + +### Bug Fixes + +* **composer.sls:** fix `salt-lint` errors ([](https://github.com/saltstack-formulas/php-formula/commit/4e48a7a)) +* **map.jinja:** fix `salt-lint` errors ([](https://github.com/saltstack-formulas/php-formula/commit/01f5ede)) +* **repo.sls:** fix `salt-lint` errors ([](https://github.com/saltstack-formulas/php-formula/commit/b4d994c)) + + +### Continuous Integration + +* merge travis matrix, add `salt-lint` & `rubocop` to `lint` job ([](https://github.com/saltstack-formulas/php-formula/commit/a5a8a95)) + ## [1.2.1](https://github.com/saltstack-formulas/php-formula/compare/v1.2.0...v1.2.1) (2019-10-07) diff --git a/FORMULA b/FORMULA index 77cefd1..d68972d 100644 --- a/FORMULA +++ b/FORMULA @@ -1,7 +1,7 @@ name: php os: Debian, Ubuntu, RedHat, Fedora, CentOS, Suse, openSUSE os_family: Debian, RedHat, Suse -version: 1.2.1 +version: 1.2.2 release: 1 minimum_version: 2017.7 summary: Php formula diff --git a/docs/AUTHORS.rst b/docs/AUTHORS.rst index 2af412d..2e45874 100644 --- a/docs/AUTHORS.rst +++ b/docs/AUTHORS.rst @@ -21,7 +21,7 @@ This list is sorted by the number of commits per contributor in *descending* ord - 48 * - :raw-html-m2r:`@myii` - `@myii `_ - - 36 + - 44 * - :raw-html-m2r:`@alxwr` - `@alxwr `_ - 31 @@ -188,4 +188,4 @@ This list is sorted by the number of commits per contributor in *descending* ord ---- -Auto-generated by a `forked version `_ of `gaocegege/maintainer `_ on 2019-10-07. +Auto-generated by a `forked version `_ of `gaocegege/maintainer `_ on 2019-10-10. diff --git a/docs/CHANGELOG.rst b/docs/CHANGELOG.rst index d7eea1a..7c5b8c7 100644 --- a/docs/CHANGELOG.rst +++ b/docs/CHANGELOG.rst @@ -2,6 +2,23 @@ Changelog ========= +`1.2.2 `_ (2019-10-10) +----------------------------------------------------------------------------------------------------- + +Bug Fixes +^^^^^^^^^ + + +* **composer.sls:** fix ``salt-lint`` errors (\ ` `_\ ) +* **map.jinja:** fix ``salt-lint`` errors (\ ` `_\ ) +* **repo.sls:** fix ``salt-lint`` errors (\ ` `_\ ) + +Continuous Integration +^^^^^^^^^^^^^^^^^^^^^^ + + +* merge travis matrix, add ``salt-lint`` & ``rubocop`` to ``lint`` job (\ ` `_\ ) + `1.2.1 `_ (2019-10-07) -----------------------------------------------------------------------------------------------------