-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat(release.config.js): use full commit hash in commit link #89
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
myii
added a commit
to saltstack-formulas/apache-formula
that referenced
this pull request
Oct 31, 2019
myii
changed the title
Feat/update release config js
WIP: feat(release.config.js): use full commit hash in commit link
Oct 31, 2019
myii
added a commit
to saltstack-formulas/iptables-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/telegraf-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/sysctl-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/zabbix-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/locale-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/redis-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/collectd-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/vim-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/golang-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/vsftpd-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/logrotate-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/fail2ban-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/dhcpd-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/mysql-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/timezone-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/rkhunter-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/php-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/users-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/systemd-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/iscsi-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/syslog-ng-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/letsencrypt-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/grafana-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/apt-cacher-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/template-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/openvpn-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/strongswan-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/deepsea-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/postfix-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/bind-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/sysstat-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/chrony-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/cert-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/openssh-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/lvm-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/vault-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/docker-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/salt-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/ntp-formula
that referenced
this pull request
Oct 31, 2019
myii
added a commit
to saltstack-formulas/rabbitmq-formula
that referenced
this pull request
Oct 31, 2019
myii
changed the title
WIP: feat(release.config.js): use full commit hash in commit link
feat(release.config.js): use full commit hash in commit link
Oct 31, 2019
* Unused, so unnecessary duplication when fixes are required
* conventional-changelog/conventional-changelog#523 - Get latest changes introduced by this PR - Fixes commit short hashes no longer showing in both the release notes and the changelog
myii
force-pushed
the
feat/update-release-config-js
branch
from
October 31, 2019 13:06
0d8bd5d
to
6313556
Compare
🎉 This PR is included in version 1.44.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
noelmcloughlin
pushed a commit
to noelmcloughlin/golang-formula
that referenced
this pull request
Nov 18, 2019
blarghmatey
pushed a commit
to mitodl/nginx-formula
that referenced
this pull request
Dec 17, 2019
markbreedlove
pushed a commit
to mitodl/nginx-formula
that referenced
this pull request
Dec 17, 2019
noelmcloughlin
pushed a commit
to noelmcloughlin/salt-formula
that referenced
this pull request
Jan 22, 2020
noelmcloughlin
pushed a commit
to noelmcloughlin/salt-formula
that referenced
this pull request
Jan 23, 2020
acalv
added a commit
to acalv/docker-formula
that referenced
this pull request
Apr 3, 2020
* passing the hold and allow_updated to docker pkg.installed * accessing correct level of config * Fix rendering error on MacOS ('oscodename') * Docker is brew name * No upstart/sysv on Darwin * chore(python): use python3 apt packages * fix(syntax): param 'refresh' replaces 'refresh_db' * Using the tplroot method (borrowed from the ufw-formula) prevents having to set --extra-filerefs when using the formula with salt-ssh. * Updating apt (ubuntu/debian) resulted in the following error: Skipping acquire of configured file 'stable/binary-i386/Packages' as repository 'https://download.docker.com/linux/ubuntu bionic InRelease' doesn't support architecture 'i386' Adding [arch=<osarch>] between deb and https solved this problem. * Updating apt (ubuntu/debian) resulted in the following error: Skipping acquire of configured file 'stable/binary-i386/Packages' as repository 'https://download.docker.com/linux/ubuntu bionic InRelease' doesn't support architecture 'i386' Adding [arch=<osarch>] between deb and https solved this problem. * Revert "Make it easier when using salt-ssh" * Use tpldir to make imports work from e.g. salt-ssh without having to resort to --extra-filerefs=salt://docker/map.jinja,... on the cmd line. Also added the os architecture to the apt sources.list line: deb [arch=<osarch>] http://... * fix(sls): avoid unnecessary rendering error * doc(clean): make remove alias for clean * fix(merge): workaround broken map.jinja merge * chore(template): align osmap with template-formula * feat(macos): docker desktop for mac support * doc(clean): make remove alias for clean * chore(template): align osmap with template-formula * feat(macos): docker desktop for mac support * pillar.example registry-mirrors * fix(install): resolve saltstack-formulas#224 * refactor(init): remove duplicated jinja * Update docker/install.sls Co-Authored-By: Imran Iqbal <myii@users.noreply.github.com> * feat(linux): archlinux support (saltstack-formulas#229) * feat(semantic-release): implement for this formula * Semi-automated using myii/ssf-formula#78 * Fix (or ignore) `yamllint` errors: ```bash docker-formula$ yamllint -s . ./pillar.example 5:1 warning missing document start "---" (document-start) 10:8 warning missing starting space in comment (comments) 12:11 error empty value in block mapping (empty-values) 13:89 error line too long (92 > 88 characters) (line-length) 14:26 warning truthy value should be one of [false, true] (truthy) 28:1 error trailing spaces (trailing-spaces) 32:11 error empty value in block mapping (empty-values) 36:89 error line too long (92 > 88 characters) (line-length) 37:26 warning truthy value should be one of [false, true] (truthy) 58:4 warning missing starting space in comment (comments) 59:4 warning missing starting space in comment (comments) 61:4 warning missing starting space in comment (comments) 62:4 warning missing starting space in comment (comments) 63:21 warning truthy value should be one of [false, true] (truthy) 64:1 error trailing spaces (trailing-spaces) 65:89 error line too long (93 > 88 characters) (line-length) 66:4 warning missing starting space in comment (comments) 76:3 warning comment not indented like content (comments-indentation) 77:6 warning missing starting space in comment (comments) 78:6 warning missing starting space in comment (comments) 79:23 warning truthy value should be one of [false, true] (truthy) 87:20 warning truthy value should be one of [false, true] (truthy) 88:20 warning truthy value should be one of [false, true] (truthy) 92:12 warning truthy value should be one of [false, true] (truthy) 111:8 warning missing starting space in comment (comments) 119:4 error syntax error: expected <block end>, but found '<block mapping start>' 120:8 warning missing starting space in comment (comments) 121:7 error wrong indentation: expected 5 but found 6 (indentation) 132:8 warning missing starting space in comment (comments) 145:2 warning missing starting space in comment (comments) 147:5 warning comment not indented like content (comments-indentation) 148:6 warning missing starting space in comment (comments) 149:6 warning missing starting space in comment (comments) 150:6 warning missing starting space in comment (comments) 154:6 warning missing starting space in comment (comments) 155:6 warning missing starting space in comment (comments) 157:2 warning missing starting space in comment (comments) 159:6 warning comment not indented like content (comments-indentation) 169:6 warning comment not indented like content (comments-indentation) 174:1 error too many blank lines (1 > 0) (empty-lines) ./docker/defaults.yaml 3:1 warning missing document start "---" (document-start) 5:22 warning truthy value should be one of [false, true] (truthy) 7:17 warning truthy value should be one of [false, true] (truthy) 14:22 warning truthy value should be one of [false, true] (truthy) 15:17 warning truthy value should be one of [false, true] (truthy) 16:10 warning truthy value should be one of [false, true] (truthy) 27:20 warning truthy value should be one of [false, true] (truthy) 28:11 warning truthy value should be one of [false, true] (truthy) 29:23 warning truthy value should be one of [false, true] (truthy) 29:35 warning missing starting space in comment (comments) 32:23 warning truthy value should be one of [false, true] (truthy) 33:14 warning truthy value should be one of [false, true] (truthy) 39:20 warning truthy value should be one of [false, true] (truthy) 40:20 warning truthy value should be one of [false, true] (truthy) 46:1 error too many blank lines (1 > 0) (empty-lines) ./docker/osfamilymap.yaml 13:2 error syntax error: found character '%' that cannot start any token 50:89 error line too long (164 > 88 characters) (line-length) ./docker/codenamemap.yaml 3:1 warning missing document start "---" (document-start) 36:1 error too many blank lines (1 > 0) (empty-lines) ./docker/osmap.yaml 1:1 error too many blank lines (1 > 0) (empty-lines) 2:1 warning missing document start "---" (document-start) (saltstack) imran@E6530:~/Salt/formulas/docker-formula$ yamllint -s . ./docker/osfamilymap.yaml 13:2 error syntax error: found character '%' that cannot start any token 51:89 error line too long (164 > 88 characters) (line-length) ``` * fix(compose-ng.sls): fix `salt-lint` errors ```bash Examining docker/compose-ng.sls of type state [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:13 {{id}}: [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:17 {{id}} image: [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:22 - name: {{image[0]}} [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:23 - tag: {{image[1]}} [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:25 - name: {{container.image}} [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:28 {{id}} container: [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:40 - name: {{id}} [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:41 - image: {{container.image}} [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:43 - command: {{container.command}} [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:48 - {{variable}}: {{value}} [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:57 - "{{mapping[0]}}" [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:59 - "{{mapping[-1]}}/tcp": [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:60 HostPort: "{{mapping[-2]}}" [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:61 HostIp: "{{mapping[-3]|d('')}}" [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:64 - {{port_mapping}} [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:71 - {{volume}} [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:78 - {{volume}} [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:86 {{name}}: {{alias}} [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:92 Name: {{policy[0]}} [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:94 MaximumRetryCount: {{policy[1]}} [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:99 - docker_image: {{id}} [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:101 - docker: {{id}} image [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:106 - docker_image: {{containerid}} [206] Jinja variables should have spaces before and after: {{ var_name }} docker/compose-ng.sls:108 - docker: {{containerid}} ``` * test(testinfra): remove from the formula * test(inspec): add tests for package, config & service * fix(pillar.example): ensure `docker.config` is available * docs(readme): move to `docs/` directory and modify accordingly * ci(travis): update `salt-lint` config for `v0.0.10` * Automated using myii/ssf-formula#82 * chore(release): 0.42.0 [skip ci] # [0.42.0](saltstack-formulas/docker-formula@v0.41.0...v0.42.0) (2019-10-23) ### Bug Fixes * **compose-ng.sls:** fix `salt-lint` errors ([](saltstack-formulas@9e8e1e8)) * **pillar.example:** ensure `docker.config` is available ([](saltstack-formulas@dce112a)) ### Continuous Integration * **travis:** update `salt-lint` config for `v0.0.10` ([](saltstack-formulas@3eaed1b)) ### Documentation * **readme:** move to `docs/` directory and modify accordingly ([](saltstack-formulas@222fc6d)) ### Features * **semantic-release:** implement for this formula ([](saltstack-formulas@ea6be11)) ### Tests * **inspec:** add tests for package, config & service ([](saltstack-formulas@451d76d)) * **testinfra:** remove from the formula ([](saltstack-formulas@62122d2)) * ci(kitchen): use `debian-10-master-py3` instead of `develop` [skip ci] * Automated using myii/ssf-formula#84 * ci(kitchen+travis): upgrade matrix after `2019.2.2` release [skip ci] * Automated using myii/ssf-formula#86 * fix(release.config.js): use full commit hash in commit link [skip ci] * Automated using myii/ssf-formula#89 * perf(travis): improve `salt-lint` invocation [skip ci] * Automated using myii/ssf-formula#90 * ci(kitchen): use `develop` image until `master` is ready (`amazonlinux`) [skip ci] * Automated using myii/ssf-formula#93 * ci(travis): use build config validation (beta) [skip ci] * Automated using myii/ssf-formula#97 * ci(travis): apply changes from build config validation [skip ci] * Automated using myii/ssf-formula#98 * ci(travis): opt-in to `dpl v2` to complete build config validation [skip ci] * Automated using myii/ssf-formula#100 * ci(travis): run `shellcheck` during lint job [skip ci] * Automated using myii/ssf-formula#106 * ci(travis): quote pathspecs used with `git ls-files` [skip ci] * Automated using myii/ssf-formula#107 * ci(gemfile): restrict `train` gem version until upstream fix [skip ci] * Automated using myii/ssf-formula#110 * ci(travis): use `major.minor` for `semantic-release` version [skip ci] * Automated using myii/ssf-formula#112 * feat: support optional container removal before start in upstart/systemd * chore(release): 0.43.0 [skip ci] # [0.43.0](saltstack-formulas/docker-formula@v0.42.0...v0.43.0) (2020-01-22) ### Bug Fixes * **release.config.js:** use full commit hash in commit link [skip ci] ([01ece3d](saltstack-formulas@01ece3d)) ### Continuous Integration * **gemfile:** restrict `train` gem version until upstream fix [skip ci] ([734d4e3](saltstack-formulas@734d4e3)) * **kitchen:** use `debian-10-master-py3` instead of `develop` [skip ci] ([d87e787](saltstack-formulas@d87e787)) * **kitchen:** use `develop` image until `master` is ready (`amazonlinux`) [skip ci] ([71c5bcb](saltstack-formulas@71c5bcb)) * **kitchen+travis:** upgrade matrix after `2019.2.2` release [skip ci] ([2189efb](saltstack-formulas@2189efb)) * **travis:** apply changes from build config validation [skip ci] ([f0a07fc](saltstack-formulas@f0a07fc)) * **travis:** opt-in to `dpl v2` to complete build config validation [skip ci] ([340556e](saltstack-formulas@340556e)) * **travis:** quote pathspecs used with `git ls-files` [skip ci] ([12bf914](saltstack-formulas@12bf914)) * **travis:** run `shellcheck` during lint job [skip ci] ([ba127a0](saltstack-formulas@ba127a0)) * **travis:** use `major.minor` for `semantic-release` version [skip ci] ([2590d61](saltstack-formulas@2590d61)) * **travis:** use build config validation (beta) [skip ci] ([fe184e9](saltstack-formulas@fe184e9)) ### Features * support optional container removal before start in upstart/systemd ([cc10d97](saltstack-formulas@cc10d97)) ### Performance Improvements * **travis:** improve `salt-lint` invocation [skip ci] ([18fa798](saltstack-formulas@18fa798)) * fix: avoid setting multiple pre-start stanzas in upstart * ci(kitchen): avoid using bootstrap for `master` instances [skip ci] * Automated using myii/ssf-formula#124 * chore(rubocop): move `LineLength` cop from `Metrics` to `Layout` [skip ci] * Automated using myii/ssf-formula#129 * chore(gitignore): update from the `template-formula` [skip ci] * Automated using myii/ssf-formula#132 Co-authored-by: Tyler Hunt <tyler.s.hunt05@gmail.com> Co-authored-by: N <noel.mcloughlin@gmail.com> Co-authored-by: N <noelmcloughlin@users.noreply.github.com> Co-authored-by: Imran Iqbal <myii@users.noreply.github.com> Co-authored-by: RemiChristiaan <remi-christiaan.cool@vbhi.com> Co-authored-by: Yoan Blanc <yoan@dosimple.ch> Co-authored-by: Imran Iqbal <iqbalmy@hotmail.com> Co-authored-by: semantic-release-bot <semantic-release-bot@martynus.net> Co-authored-by: Chad Bean <chad.bean@solarwinds.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
and the changelog
To be merged after #86.