Skip to content
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

fix lsp-types benchmark #3079

Merged
merged 8 commits into from
Aug 15, 2022
Merged

fix lsp-types benchmark #3079

merged 8 commits into from
Aug 15, 2022

Conversation

pepeiborra
Copy link
Collaborator

@pepeiborra pepeiborra commented Aug 6, 2022

The lsp-types benchmark definition needs some fixes in ghc 9.2

Also fixes #3102 by tar'ing the benchmark artefacts before upload

@pepeiborra pepeiborra changed the title fix lsp-types paths fix lsp-types benchmark Aug 6, 2022
@pepeiborra pepeiborra marked this pull request as ready for review August 7, 2022 08:25
@pepeiborra pepeiborra requested a review from kokobd August 7, 2022 08:25
ghcide/bench/lib/Experiments.hs Outdated Show resolved Hide resolved
@michaelpj
Copy link
Collaborator

Maybe we should switch to a more stable package for benchmarking 😆

@pepeiborra
Copy link
Collaborator Author

Maybe we should switch to a more stable package for benchmarking 😆

True, I chose lsp-types because it is guaranteed to be buildable by our CI, uses TH, and doesn't increase our dependency closure, but there are other options like aeson that also check those boxes

@pepeiborra pepeiborra merged commit e55004a into master Aug 15, 2022
sloorush pushed a commit to sloorush/haskell-language-server that referenced this pull request Sep 12, 2022
* fix lsp-types paths

* Fail if there are no documents to benchmark

* use modules with suitable positions

* Fix 8.6.5 build

* fix imports for 8.6.5

* Fix benchmark artifact uploads

* format
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI bench-unit job failures
3 participants