-
Notifications
You must be signed in to change notification settings - Fork 670
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
Fix backward compatibility #3929
Conversation
I basically made a symlink ➜ .config git:(main) ls -l | grep requirements | grep lock
-rw-r--r-- 1 user staff 1014 Dec 5 15:36 lock-requirements.txt
lrwxr-xr-x 1 user staff 21 Dec 5 15:37 requirements-lock.txt -> lock-requirements.txt |
@ssbarnea I think, we should update the file name for |
@ajinkyau
But if you pull my changes to your local machine, you will see it's a symlink: $ .config git:(main) ls -l | grep requirements | grep lock
-rw-r--r-- 1 user staff 1014 Dec 5 15:36 lock-requirements.txt
lrwxr-xr-x 1 user staff 21 Dec 5 15:37 requirements-lock.txt -> lock-requirements.txt note: |
Okey, this view represents better, what I just did |
This PR contains the following updates: | Package | Update | Change | |---|---|---| | [ansible-lint](https://github.com/ansible/ansible-lint) ([changelog](https://github.com/ansible/ansible-lint/releases)) | patch | `==6.22.1` -> `==6.22.2` | --- ### Release Notes <details> <summary>ansible/ansible-lint (ansible-lint)</summary> ### [`v6.22.2`](https://github.com/ansible/ansible-lint/releases/tag/v6.22.2) [Compare Source](ansible/ansible-lint@v6.22.1...v6.22.2) #### Bugfixes - Fix key error for name\[casing] rule ([#​3987](ansible/ansible-lint#3987)) [@​ajinkyau](https://github.com/ajinkyau) - Allow dashes in legacy role namespaces ([#​3962](ansible/ansible-lint#3962)) [@​sur5r](https://github.com/sur5r) - Use new ansible-compat verbosity levels ([#​3975](ansible/ansible-lint#3975)) [@​ssbarnea](https://github.com/ssbarnea) - Remove dependency on newer requests library ([#​3959](ansible/ansible-lint#3959)) [@​ssbarnea](https://github.com/ssbarnea) - Ignore set-property for systemd command ([#​3949](ansible/ansible-lint#3949)) [@​alanbbr](https://github.com/alanbbr) - Correct requires_ansible error message ([#​3954](ansible/ansible-lint#3954)) [@​ssbarnea](https://github.com/ssbarnea) - Improve transformation for `no-free-form` rule ([#​3945](ansible/ansible-lint#3945)) [@​ajinkyau](https://github.com/ajinkyau) - Documentation improvement ([#​3946](ansible/ansible-lint#3946)) [@​ssbarnea](https://github.com/ssbarnea) - docs: fix grammatical issue in philosophy Q\&A section ([#​3934](ansible/ansible-lint#3934)) [@​davidhulick](https://github.com/davidhulick) - Update supported versions of ansible ([#​3930](ansible/ansible-lint#3930)) [@​ajinkyau](https://github.com/ajinkyau) - Fix backward compatibility ([#​3929](ansible/ansible-lint#3929)) [@​McSim85](https://github.com/McSim85) - Fix auto capitalization for name\[prefix] rule ([#​3922](ansible/ansible-lint#3922)) [@​ajinkyau](https://github.com/ajinkyau) - Fix role deps check for detecting path names ([#​3923](ansible/ansible-lint#3923)) [@​cavcrosby](https://github.com/cavcrosby) - Avoid warnings about PATH with pipx installations ([#​3920](ansible/ansible-lint#3920)) [@​ssbarnea](https://github.com/ssbarnea) </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xNDAuMCIsInVwZGF0ZWRJblZlciI6IjM3LjE0MC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9--> Reviewed-on: https://git.home/nrdufour/home-ops/pulls/319 Co-authored-by: Renovate <renovate@ptinem.io> Co-committed-by: Renovate <renovate@ptinem.io>
Any versions before this PR #3762 (
v6.22.1
) use themain
branch forrequirements-lock.txt
. (full urlhttps://raw.githubusercontent.com/ansible/ansible-lint/main/.config/requirements-lock.txt
)It still uses the main branch even if you specify a particular version.
For example, this config
will try to download the main branch:
Run wget --output-file=.git/ansible-lint-requirements.txt https://raw.githubusercontent.com/ansible/ansible-lint/main/.config/requirements-lock.txt wget --output-file=.git/ansible-lint-requirements.txt https://raw.githubusercontent.com/ansible/ansible-lint/main/.config/requirements-lock.txt shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0} Error: Process completed with exit code 8.
The PR that broke backward compatibility is this one #3925
Fixes: #3928
7fa5f76