Delete the Brittany plugin (#3441)

It's unmaintained and not popular.

Fixes #3346.

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

Author: michaelpj

.github/workflows/hackage.yml

@@ -29,7 +29,7 @@ jobs:
         package: ["hie-compat", "hls-graph", "shake-bench",
                   "hls-plugin-api", "ghcide", "hls-test-utils",
                   "hls-cabal-plugin",
-                  "hls-brittany-plugin", "hls-floskell-plugin", "hls-fourmolu-plugin",
+                  "hls-floskell-plugin", "hls-fourmolu-plugin",
                   "hls-ormolu-plugin", "hls-stylish-haskell-plugin",
                   "hls-class-plugin", "hls-eval-plugin", "hls-explicit-imports-plugin",
                   "hls-haddock-comments-plugin", "hls-hlint-plugin", "hls-stan-plugin",
@@ -106,14 +106,11 @@ jobs:
       # These tweaks are already in cabal-901.project but we dont want to use the entire file,
       # Only the tricks needed by the solver which we know will not make the hackage build fail.
       # The solver takes in account all project packages, even if they are not gonna be effectively built
-      # (like brittany or stylish-haskell for ghc-9.0)
+      # (like stylish-haskell for ghc-9.0)
       - name: "Add temporary needed allow-newer for ghc-9.0"
         if: steps.get-hackage-version.outputs.exists != 'true' && matrix.ghc == '9.0.2'
         run: |
           cd $(ls -d ./incoming/${{ matrix.package }}-*)
-          # For brittany
-          echo "allow-newer:" >> cabal.project
-          echo " butcher:base, multistate:base, data-tree-print:base," >> cabal.project
           # For stylish-haskell
           echo " stylish-haskell:Cabal,stylish-haskell:ghc-lib-parser,stylish-haskell:aeson" >> cabal.project
 

.github/workflows/test.yml

@@ -165,10 +165,6 @@ jobs:
           HLS_WRAPPER_TEST_EXE: hls-wrapper
         run: cabal test wrapper-test --test-options="$TEST_OPTS --rerun-log-file .tasty-rerun-log-wrapper"
 
-      - if: matrix.test && matrix.ghc != '9.2.4' && matrix.ghc != '9.2.5' && matrix.ghc != '9.4.2' && matrix.ghc != '9.4.3'
-        name: Test hls-brittany-plugin
-        run: cabal test hls-brittany-plugin --test-options="$TEST_OPTS" || LSP_TEST_LOG_COLOR=0 LSP_TEST_LOG_MESSAGES=true LSP_TEST_LOG_STDERR=true cabal test hls-brittany-plugin --test-options="$TEST_OPTS"
-
       - if: matrix.test
         name: Test hls-refactor-plugin
         run: cabal test hls-refactor-plugin --test-options="$TEST_OPTS" || LSP_TEST_LOG_COLOR=0 LSP_TEST_LOG_MESSAGES=true LSP_TEST_LOG_STDERR=true cabal test hls-refactor-plugin --test-options="$TEST_OPTS"

CODEOWNERS

@@ -8,7 +8,6 @@
 
 # Plugins
 /plugins/hls-alternate-number-format-plugin @drsooch
-/plugins/hls-brittany-plugin @fendor
 /plugins/hls-cabal-plugin @fendor
 /plugins/hls-cabal-fmt-plugin @VeryMilkyJoe @fendor
 /plugins/hls-call-hierarchy-plugin @July541

bench/config.yaml

@@ -144,7 +144,6 @@ configurations:
   - rename
   - stylish-haskell
 # - alternateNumberFormat
-# # - brittany
 # - callHierarchy
 # - changeTypeSignature
 # - class

cabal.project

@@ -11,7 +11,6 @@ packages:
          ./plugins/hls-cabal-plugin
          ./plugins/hls-cabal-fmt-plugin
          ./plugins/hls-tactics-plugin
-         ./plugins/hls-brittany-plugin
          ./plugins/hls-stylish-haskell-plugin
          ./plugins/hls-fourmolu-plugin
          ./plugins/hls-class-plugin

configuration-ghc-90.nix

@@ -1,7 +1,7 @@
 { pkgs, inputs }:
 
 let
-  disabledPlugins = [ "hls-brittany-plugin" "hls-stylish-haskell-plugin" ];
+  disabledPlugins = [ "hls-stylish-haskell-plugin" ];
 
   hpkgsOverride = hself: hsuper:
     with pkgs.haskell.lib;
@@ -34,7 +34,7 @@ let
       # Re-generate HLS drv excluding some plugins
       haskell-language-server =
         hself.callCabal2nixWithOptions "haskell-language-server" ./.
-        (pkgs.lib.concatStringsSep " " [ "-f-brittany" "-f-stylishhaskell" ])
+        (pkgs.lib.concatStringsSep " " [ "-f-stylishhaskell" ])
         { };
 
       retrie = hself.retrie_1_1_0_0;

docs/configuration.md

@@ -40,7 +40,7 @@ This option obviously would not make sense for language servers for other langua
 
 Here is a list of the additional settings currently supported by `haskell-language-server`, along with their setting key (you may not need to know this) and default:
 
-- Formatting provider (`haskell.formattingProvider`, default `ormolu`): what formatter to use; one of `floskell`, `ormolu`, `fourmolu`, `stylish-haskell`, or `brittany` (if compiled with the brittany plugin).
+- Formatting provider (`haskell.formattingProvider`, default `ormolu`): what formatter to use; one of `floskell`, `ormolu`, `fourmolu`, or `stylish-haskell`.
 - Max completions (`haskell.maxCompletions`, default 40): maximum number of completions sent to the LSP client.
 - Check project (`haskell.checkProject`, default true): whether to typecheck the entire project on initial load. As it is activated by default could drive to bad performance in large projects.
 - Check parents (`haskell.checkParents`, default `CheckOnSave`): when to typecheck reverse dependencies of a file; one of `NeverCheck`, `CheckOnSave` (means dependent/parent modules will only be checked when you save), or `AlwaysCheck` (means re-typechecking them on every change).

docs/contributing/plugin-tutorial.md

@@ -77,7 +77,7 @@ The HLS codebase includes several plugins under the namespace `Ide.Plugin.*`, th
 
 - The ghcide plugin, which embeds ghcide as a plugin (ghcide is also the engine under HLS).
 - The example and example2 plugins, offering a dubious welcome to new contributors
-- The Brittany, ormolu, fourmolu, floskell and stylish-haskell plugins, a testament to the code formatting wars of our community.
+- The ormolu, fourmolu, floskell and stylish-haskell plugins, a testament to the code formatting wars of our community.
 - The eval plugin, a code lens provider to evaluate code in comments
 - The retrie plugin, a code actions provider to execute retrie commands
 
@@ -96,7 +96,6 @@ idePlugins = pluginDescToIdePlugins allPlugins
       , Ormolu.descriptor "ormolu"
       , StylishHaskell.descriptor "stylish-haskell"
       , Retrie.descriptor "retrie"
-      , Brittany.descriptor "brittany"
       , Eval.descriptor "eval"
       ]
 ```

docs/features.md

@@ -107,7 +107,6 @@ Format your code with various Haskell code formatters.
 
 | Formatter       | Provided by                  |
 | --------------- | ---------------------------- |
-| Brittany        | `hls-brittany-plugin`        |
 | Floskell        | `hls-floskell-plugin`        |
 | Fourmolu        | `hls-fourmolu-plugin`        |
 | Ormolu          | `hls-ormolu-plugin`          |

docs/support/plugin-support.md

@@ -63,7 +63,6 @@ For example, a plugin to provide a formatter which has itself been abandoned has
 | `hls-refine-imports-plugin`         | 2    |                          |
 | `hls-stylish-haskell-plugin`        | 2    | 9.4                      |
 | `hls-tactics-plugin`                | 2    | 9.2, 9.4                 |
-| `hls-brittany-plugin`               | 3    | 9.2, 9.4                 |
 | `hls-haddock-comments-plugin`       | 3    | 9.2, 9.4                 |
 | `hls-stan-plugin`                   | 3    | 8.6, 9.0, 9.2, 9.4       |
 | `hls-retrie-plugin`                 | 3    |                          |