-
Notifications
You must be signed in to change notification settings - Fork 527
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
Add flymake-ansible-lint.el to list of tools #2239
Conversation
Thanks for your Ansible docs contribution! We talk about Ansible documentation on Matrix at #docs:ansible.im if you ever want to join us and chat about the docs! We meet on Matrix every Tuesday. See the Ansible calendar for meeting details. We welcome additions to our weekly agenda items too. You can add the |
@@ -25,6 +25,7 @@ A free, open-source text editor and IDE that supports auto-indentation, syntax h | |||
* `jinja2-mode <https://github.com/paradoxxxzero/jinja2-mode>`_ - Jinja2 highlighting and syntax checking. | |||
* `magit-mode <https://github.com/magit/magit>`_ - Git porcelain within Emacs. | |||
* `lsp-mode <https://emacs-lsp.github.io/lsp-mode/page/lsp-ansible/>`_ - Ansible syntax highlighting, auto-completion and diagnostics. | |||
* `flymake-ansible-lint <https://github.com/jamescherti/flymake-ansible-lint.el>`_ - Ansible Lint integration with automatic/continuous annotation of errors, warnings, and info while editing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@omsai This looks OK to me. Thanks for the contribution. Could you please amend the commit message to something a bit more meaningful than fixes Ansible lint?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@oraNod Indeed, that was sloppy of me 😅 - is the new commit message okay?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No worries @omsai that looks good to me now. Thanks again for adding this to the docs.
Backport to stable-2.16: 💚 backport PR created✅ Backport PR branch: Backported as #2275 🤖 @patchback |
Backport to stable-2.17: 💚 backport PR created✅ Backport PR branch: Backported as #2276 🤖 @patchback |
Backport to stable-2.18: 💚 backport PR created✅ Backport PR branch: Backported as #2278 🤖 @patchback |
…2239) (#2276) (cherry picked from commit 2cf2053) Co-authored-by: Pariksheet Nanda <[email protected]>
…2239) (#2275) (cherry picked from commit 2cf2053) Co-authored-by: Pariksheet Nanda <[email protected]>
…2239) (#2278) (cherry picked from commit 2cf2053) Co-authored-by: Pariksheet Nanda <[email protected]>
Using
ansible-lint
inside of Emacs available using a package from MELPA ansible/ansible-lint#1326