branch: elpa/flycheck
commit dd41def48472f3f0a03798211f5136a1195d25fc
Author: Bozhidar Batsov <[email protected]>
Commit: Bozhidar Batsov <[email protected]>
Remove yaml-ruby checker
Ruby's YAML parser provides the same functionality as js-yaml
(yaml-jsyaml), which is more commonly installed. yaml-yamllint
remains available for linting.
---
doc/languages.rst | 6 +-----
flycheck.el | 21 ---------------------
test/specs/languages/test-yaml.el | 9 +--------
3 files changed, 2 insertions(+), 34 deletions(-)
diff --git a/doc/languages.rst b/doc/languages.rst
index 31d1bc11f2..a53a55ccfd 100644
--- a/doc/languages.rst
+++ b/doc/languages.rst
@@ -1662,7 +1662,7 @@ to view the docstring of the syntax checker. Likewise,
you may use
.. supported-language:: YAML
- Flycheck checks YAML with `yaml-actionlint`, `yaml-jsyaml`, `yaml-ruby` or
`yaml-yamllint`.
+ Flycheck checks YAML with `yaml-actionlint`, `yaml-jsyaml` or
`yaml-yamllint`.
.. syntax-checker:: yaml-actionlint
@@ -1672,10 +1672,6 @@ to view the docstring of the syntax checker. Likewise,
you may use
Check syntax with `js-yaml <https://github.com/nodeca/js-yaml>`_.
- .. syntax-checker:: yaml-ruby
-
- Check syntax with Ruby's YAML parser.
-
.. syntax-checker:: yaml-yamllint
Check syntax with :program:`yamllint`.
diff --git a/flycheck.el b/flycheck.el
index 88d5048e79..5dd2acd90e 100644
--- a/flycheck.el
+++ b/flycheck.el
@@ -243,7 +243,6 @@
xml-xmllint
yaml-actionlint
yaml-jsyaml
- yaml-ruby
yaml-yamllint)
"Syntax checkers available for automatic selection.
@@ -12767,26 +12766,6 @@ See URL `https://github.com/nodeca/js-yaml'."
:next-checkers ((warning . yaml-yamllint)
(warning . cwl)))
-(flycheck-define-checker yaml-ruby
- "A YAML syntax checker using Ruby's YAML parser.
-
-This syntax checker uses the YAML parser from Ruby's standard
-library.
-
-See URL `https://www.ruby-doc.org/stdlib-2.0.0/libdoc/yaml/rdoc/YAML.html'."
- :command ("ruby" "-ryaml" "-e" "begin;
- YAML.load(STDIN); \
- rescue Exception => e; \
- STDERR.puts \"stdin:#{e}\"; \
- end")
- :standard-input t
- :error-patterns
- ((error line-start "stdin:" (zero-or-more not-newline) ":" (message)
- "at line " line " column " column line-end))
- :modes (yaml-mode yaml-ts-mode)
- :next-checkers ((warning . yaml-yamllint)
- (warning . cwl)))
-
(flycheck-def-config-file-var flycheck-yamllintrc
yaml-yamllint
'(".yamllint"
diff --git a/test/specs/languages/test-yaml.el
b/test/specs/languages/test-yaml.el
index 7b35d55302..e670b7d3ad 100644
--- a/test/specs/languages/test-yaml.el
+++ b/test/specs/languages/test-yaml.el
@@ -34,15 +34,8 @@
'(4 5 error "bad indentation of a mapping entry"
:checker yaml-jsyaml)))
- (flycheck-buttercup-def-checker-test yaml-ruby yaml nil
- (let ((flycheck-disabled-checkers '(yaml-jsyaml)))
- (flycheck-buttercup-should-syntax-check
- "language/yaml.yaml" 'yaml-mode
- '(4 5 error "mapping values are not allowed in this context"
- :checker yaml-ruby))))
-
(flycheck-buttercup-def-checker-test yaml-yamllint yaml nil
- (let ((flycheck-disabled-checkers '(yaml-yamlyaml)))
+ (let ((flycheck-disabled-checkers '(yaml-jsyaml)))
(flycheck-buttercup-should-syntax-check
"language/yaml.yaml" 'yaml-mode
'(3 1 warning "missing document start \"---\" (document-start)"