Remove XML formatter

As was mentioned in: #372 (comment) ...this formatter doesn't appear to be used, and is superseded by the `Checkstyle` plugin. Removing it allows us to remove a dependency on Nokogiri, which significantly speeds up test runs in Travis since it does not need to be installed. It also removes a source of confusion, as some developers mistake it for the `Checkstyle` plugin at first.
sds · May 11, 2015 · c6df8d7 · c6df8d7
1 parent cbc7047
commit c6df8d7
Show file tree

Hide file tree

Showing 6 changed files with 3 additions and 158 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -9,6 +9,9 @@
 * Load configuration file from user's home directory if one exists and no other
   configuration file is specified or in the current working directory
 * Improve issue reporting instructions displayed when scss-lint crashes
+* Remove `XML` formatter (see the
+  [`Checkstyle` formatter](https://github.com/Sweetchuck/scss_lint_reporter_checkstyle)
+  for an XML-based alternative)
 
 ## 0.38.0
 

diff --git a/README.md b/README.md
@@ -268,26 +268,6 @@ Outputs JSON with filenames and an array of issue objects.
 }
 ```
 
-### XML
-
-Outputs XML with `<lint>`, `<file>`, and `<issue>` tags.
-
-```bash
-scss-lint --format=XML [scss-files...]
-```
-
-```xml
-<?xml version="1.0" encoding="utf-8"?>
-<lint>
-  <file name="test.css">
-    <issue line="2" severity="warning" reason="Prefer single quoted strings" />
-    <issue line="2" severity="warning" reason="Line should be indented 0 spaces, but was indented 1 spaces" />
-    <issue line="5" severity="warning" reason="Prefer single quoted strings" />
-    <issue line="6" severity="warning" reason="URLs should be enclosed in quotes" />
-  </file>
-</lint>
-```
-
 ### Plugins
 
 There are also formatters that integrate with third-party tools which are available as plugins.

diff --git a/lib/scss_lint/reporter/xml_reporter.rb b/lib/scss_lint/reporter/xml_reporter.rb
diff --git a/scss_lint.gemspec b/scss_lint.gemspec
@@ -27,6 +27,5 @@ Gem::Specification.new do |s|
   s.add_dependency 'rainbow', '~> 2.0'
   s.add_dependency 'sass', '~> 3.4.1'
 
-  s.add_development_dependency 'nokogiri', '~> 1.6.0'
   s.add_development_dependency 'rspec', '~> 3.0'
 end
diff --git a/spec/scss_lint/reporter/xml_reporter_spec.rb b/spec/scss_lint/reporter/xml_reporter_spec.rb
diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
@@ -1,5 +1,4 @@
 require 'scss_lint'
-require 'nokogiri'
 
 Dir[File.dirname(__FILE__) + '/support/**/*.rb'].each { |f| require f }