Skip to content
This repository was archived by the owner on Oct 7, 2020. It is now read-only.

Remove ide-backend #179

Merged
merged 2 commits into from
Jan 18, 2016
Merged

Conversation

cocreature
Copy link
Collaborator

There are two reasons for this:

  1. It causes our travis builds to run out of memory.
  2. ide-backend doesn’t really provide anything which we don’t already
    have and it doesn’t fit in very well (e.g. it can’t reuse ghc sessions).

There are two reasons for this:
1. It causes our travis builds to run out of memory.
2. ide-backend doesn’t really provide anything which we don’t already
have and it doesn’t fit in very well (e.g. it can’t reuse ghc sessions).
.PHONY: test-haskell
test-haskell:
# stack build --test --pedantic
# stack complains about deprecations in ghc-mod as an extra dep
# build with -Werror enabled but run the tests without it because ide-backend picks it up
stack build --test
stack build --test --flag haskell-ide-engine:pedantic \
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks encouraging

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should call pedantic.sh here instead, so the flags only get set in one place

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You’re right, but it requires some bash to not pass +RTS -N1 -RTS here. I’ll look the syntax for that up tomorrow and change it.

cocreature added a commit that referenced this pull request Jan 18, 2016
@cocreature cocreature merged commit 164c038 into haskell:master Jan 18, 2016
@mgsloan
Copy link
Collaborator

mgsloan commented Jan 18, 2016

Seems reasonable to me. My plan for the ide-backend based information was to have it be independent of the ide-backend package itself (just based on the code).

This code is still interesting to reference, thanks for writing it!

@cocreature
Copy link
Collaborator Author

@mgsloan I’m fine with moving some code over, in the case of type info I am not sure it’s very useful to have a type info plugin based on ide-backend and one on ghc-mod, but if there are useful differences, I’m happy to include something like that. It would be great if you could make a list of the things you’d like to have from ide-backend, then we can see how we provide them in a nice way.

@alanz alanz added this to the prehistory milestone Feb 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants