-
-
Notifications
You must be signed in to change notification settings - Fork 369
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
More completions fixes #2354
Merged
Merged
More completions fixes #2354
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jneira
approved these changes
Nov 15, 2021
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, thanks for continue improving completions
jneira
added a commit
to jneira/haskell-language-server
that referenced
this pull request
Nov 17, 2021
jneira
added a commit
that referenced
this pull request
Nov 18, 2021
* Use hackage version of czipwith * Update stack.yaml's * Remove source czipwith * Update index and clean cabal-ghc921.project * extra-1.7.9 has a breaking change so no way * Increase resources for circleci As for November 16, circleci has set a timeout of 1 hour for jobs in free plans Trying to increase resources to make the build faster as suggested by its support center * Use more threads reduce build times * Correct runs-on field * Update stack.yaml's * Dont fail fast if we are checking * Use ref_name to check it is a check * Bump patch version of brittany plugin * Allow newer base/th for czipwith * Add enough allow-newers to allow build the project Having all subpackages included in the `packages` field * Correct allow-newer's * Add missing cd's * Bump up ghcide to 1.5.0 * Exclude pkgs not buildable with 9.0.1 * Bum up plugins with changes versions * Bum up subpackages with changes versions * Bum up hls version to 1.5.0 * Bump up haddock plugin version * Allow ghcide 1.5 * Use ghcide 1.5.0 in the rest of plugins * Use ghcide 1.5.0 for pragmas * Allow tactics 1.5.0 for hls * Remove czipwith in hackage ci script * Disable 9.0.1 for hackage ci * Use bounds operator consistently * Build formatters with newer ghc-lib-parser * Only stylish-haskell needs newer ghc-lib * Avoid ghc-9.0.1 in other way * Script needs a min time version * First draft of 1.5.0 changelog * Comment out hackage for 9.0.1 * allow newer ghc-lib-parser * Remove redundant entry * Correct condition * Cancel previous hackage jobs * Add link to pepe's talk slides * Use th-extras master * Use th-extras master and uncomment packages * add extra-1.7.9 * Add required extra-deps * Comment unbuildable packages again * Add ignore-plugins-ghc-bounds * Add #2354 * Complete ghc deprecation notice * Add extra-1.7.9 * Add #2346 to Changelog
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We rely on local completions "shadowing" global ones and preventing redundant imports from being inserted (#2270). However, local completions require typechecking to compute, and because we use stale data to produce completions, there can be still some corner cases in which redundant import statements can be inserted if local completions stale data was not available. This should be rare