* Add ubuntu builds to gitlab CI

Fixes #3169

* Run pipelines from web interface

* Use nix for darwin

* Drop 9.2.3 and add 9.4.3

* Build in parallel

* Add 9.2.5

* fixes

* Fix x86_64-darwin

* Drop 9.4.3 (ghc-exactprint)

* Revert "Drop 9.4.3 (ghc-exactprint)"

This reverts commit 9f0936d5.

* Bump index state to allow 9.4.3

* debug

* fixes

* fixes

* fixes

* fixes

* fixes

* fixes

Co-authored-by: Julian Ospald <hasufell@posteo.de>

Jakob Brünker authored 2 years ago and Zubin committed 2 years ago

In 9.4, the ability for parser source plugins to access and manipulate
non-fatal parse errors was added:
https://gitlab.haskell.org/ghc/ghc/-/wikis/migration/9.4#parser-plugins-have-a-different-type

HLS always threw an error in this situation without running the plugins
though. This commit fixes that.

Jakob Brünker authored 2 years ago and Zubin committed 2 years ago

Since ghc 9.0, plugins are stored in the HscEnv, not in the DynFlags.
This caused HLS not to run source plugins anymore. This commit fixes
that.

Fixes #3299.

Zubin authored 2 years ago and Zubin committed 2 years ago

Storing rendered names as `Text`, especially for parents, adds a lot of duplication to the ExportsMap.
Instead we store the `OccName`s directly, which have hash-consed symbols due stored as `FastStrings`
and render it out on demand (which is just decoding the UTF-8 FastString to UTF-16 text for text <2.0,
and essentially free on text >2.0).

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

* Initial working version

* Cleanup, some compat work

* Update mixed test to include an unused field

* Remove redundant imports, add a test case

* Reorganize name related functions

* Build name map instead of walking through list

* Refactor fromJust

* Make it compatible with GHC 9.2

* Fix import conflicts

* Make it compatible with GHC 9.4

* Add missing test files

* Improve documentation, use UniqFM

* Handle maybe without 'fail'ing

* Add UniqFM compat

* Fix import list

* Make name collection its own rule

* Remove LocatedN stuff, reorganize some docs

Zubin authored 2 years ago and Zubin committed 2 years ago

Wait for index instead of compliation before making requests
as indexing happens in the background in parallel with everything
else, so we have to synchronize on the database being ready before
making any call hierarchy requests.

* fix fourmolu with -f-fixity-th in nix env

* ci failure test

* bump nix file.

* disable cachix action

* further comment out

* revive cachix action.

* remove commented-out

* fix hls-cabal-plugin by removing Cabal-syntax.
note that this is an opposite to #3383, to prioritize CI fix first.

* limit implicit-hie < 0.1.3

* added a small comment and link to the issue.

* put implicit-hie to ghcide dep

* bump CI

* fix fourmolu with -f-fixity-th in nix env

* added a small comment and link to the issue.

Co-authored-by: Michael Peyton Jones <me@michaelpj.com>

* Add CI for 9.4.3

* exclude in Windows

* Workaround fourmolu 0.9.0.0 build failure in windows

Bumps [technote-space/get-diff-action](https://github.com/technote-space/get-diff-action) from 6.1.1 to 6.1.2.
- [Release notes](https://github.com/technote-space/get-diff-action/releases)
- [Changelog](https://github.com/technote-space/get-diff-action/blob/main/.releasegarc)
- [Commits](https://github.com/technote-space/get-diff-action/compare/v6.1.1...v6.1.2

)

---
updated-dependencies:
- dependency-name: technote-space/get-diff-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* Support initial generic config for plugins

This enables opt-in  plugins, i.e. plugins that are not enabled by default

* Support initial generic config for plugins

This enables opt-in  plugins, i.e. plugins that are not enabled by default

* Address review comments and remove optionality in configInitialGenericConfig

I can't remember why this was optional in the first place, which probably means no good reason for it to be

* use defConfig instead of def

* test

* descriptor is the golden source for plugin default configs

Read plugin defaults from plugin descriptor instead of copying into initial config

This is less fragile as now the initial generic config specified in the plugin
descriptor is enforced by construction.

The downside is that now the descriptor is needed to parse the config, which
means a few places where a plugin id was enough now require a plugin descriptor,
which is much harder to get. We paper over this by moving these call sites to
the Action monad.

* redundant import

* simplify

* Build fixes

* Build fixes

* pedantic fixes

Bumps [technote-space/get-diff-action](https://github.com/technote-space/get-diff-action) from 6.1.0 to 6.1.1.
- [Release notes](https://github.com/technote-space/get-diff-action/releases)
- [Changelog](https://github.com/technote-space/get-diff-action/blob/main/.releasegarc)
- [Commits](https://github.com/technote-space/get-diff-action/compare/v6.1.0...v6.1.1

)

---
updated-dependencies:
- dependency-name: technote-space/get-diff-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Michael Peyton Jones <me@michaelpj.com>
Co-authored-by: Kobayashi <contact@zelinf.net>
Co-authored-by: Pepe Iborra <pepeiborra@gmail.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>

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

The table is skewed after the change that adds 9.2.5

Bumps [fkirc/skip-duplicate-actions](https://github.com/fkirc/skip-duplicate-actions) from 5.2.0 to 5.3.0.
- [Release notes](https://github.com/fkirc/skip-duplicate-actions/releases)
- [Commits](https://github.com/fkirc/skip-duplicate-actions/compare/v5.2.0...v5.3.0

)

---
updated-dependencies:
- dependency-name: fkirc/skip-duplicate-actions
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Michael Peyton Jones <me@michaelpj.com>

* Add CI flows for GHC 9.2.5

* Prefer GHC 9.2.5 over 9.2.4

* Update list of tested-with GHC versions, as we only *test* 9.2.5

* Add GHC 9.2.5 as a supported GHC version

* Update comment in cabal-plugin about disabled test-case

* Disable GHC 9.2.5 on windows

* Fix CI for windows

* Run tests on windows for GHC 9.2.4

Co-authored-by: Michael Peyton Jones <me@michaelpj.com>
Co-authored-by: Pepe Iborra <pepeiborra@gmail.com>

* wingman: make a copy of wingman in new directory

* wip

Co-authored-by: Santiago Weight <santiago.weight@lmns.com>

* Make redundant import removal work on patsyn imports

* Make stylish-haskell happy

Co-authored-by: Michael Peyton Jones <me@michaelpj.com>

* Remove dead and obsolete code

* Organise export list with respect to utility functions

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