Use implicit-hie cradle

[Avi-D-coder]

Companion to #110

[Avi-D-coder]

There's a issue with stack and exe/Main.hs Just took a minute to load.

[Avi-D-coder]

Followup to this would be preferred cabal or stack config. I thought about doing this via hie.yaml, but that's the wrong approach it should be a LSP config, since hie.yamls are often committed.

Is it easy to make cabal-helper follow this config? I have not yet looked into it.

[fendor]

cabal-helper can implement this behaviour easily. Just filter for the project context, if there is none report an error, otherwise use the context

[Avi-D-coder]

In light of yet another reddit post by someone confused by hie.yaml.

It's better then nothing

I would like to push for this to be merged, it does not always generate the exact right config, but it works for ever config a beginner would use. When it fails, the hie-bios no cradle matched error message is decipherable. When using conditionals or common stanzas, more advanced users will still have to write an hie.yaml, but they will have a good error message and a tool implicit-hie to help them.

There are better solutions in the pipeline, but that should not stop us from improving user experience now. Let's not let perfect be the enemy of good.

[jneira]

Agree, i thought a more complete version would be ready sooner.
However i think that it would need a specific section in the readme, specifying in which known cases the implicit config will not work for sure.

[fendor]

Unneeded, afaict?

[Avi-D-coder]

Needed for haskell/hie-bios@a852cff

[fendor]

Thank you!
Please rebase, maybe some words in the README how your tool can be used to generate a hie.yaml config

[fendor]

Also, the wrapper needs to be modified, otherwise it will continue to use cabal-helper.
The change should be of the same size.

[Avi-D-coder]

I'll rebase, when I get a chance hopefully tomorrow, very busy.

[Avi-D-coder]

@fendor I'm in the process of rebasing, but hie-bios has changed, how soon before the next hie-bios release. Unless we want hls to depend on an out of date, but still not published hie-bio commit we need to wait until hls is using a newer hie-bios.

[fendor]

@Avi-D-coder I want to hopefully do a release this week, but there are three pending PRs that need to be reviewed and merged. So, unfortunately, no promises.
Maybe we can do a minor release before the major release.

[alanz]

@fendor @Avi-D-coder what is holding this up? I think it is important we get it in.

[fendor]

@alanz A hie-bios release that includes: haskell/hie-bios@a852cff
I will get to it tomorrow.

[Avi-D-coder]

New version of hie-bios needs to be released, ghcide & hls need to use it, and then I need to find time to rebase and fix the wrapper.

[alanz]

Ok, so it is waiting, rather than abandoned. Thanks.

[Avi-D-coder]

Rebased at #186