Skip to content

add ghcide-bench flag to .cabal file#4542

Merged
fendor merged 4 commits into
haskell:masterfrom
juhp:patch-2
Apr 13, 2025
Merged

add ghcide-bench flag to .cabal file#4542
fendor merged 4 commits into
haskell:masterfrom
juhp:patch-2

Conversation

@juhp

@juhp juhp commented Apr 3, 2025

Copy link
Copy Markdown
Contributor

Closes #4544

@fendor fendor left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. Which issue does this close again?

@juhp

juhp commented Apr 4, 2025

Copy link
Copy Markdown
Contributor Author

Okay I created #4544: sorry if I didn't follow the proper workflow.
Do I need to reference the issue from the commit?

Also I wish this flag (as well as test-exe) would default to False: I didn't do this since I would rather not touch the CI, if you are using them in CI?? If not then they should definitely be off IMO.

(BTW might be good to have a PULL_REQUEST_TEMPLATE.md with a short checklist if you require certain things in PRs, just a suggestion :-)

@juhp

juhp commented Apr 4, 2025

Copy link
Copy Markdown
Contributor Author

Ugh why didn't github rebase...

@fendor

fendor commented Apr 4, 2025

Copy link
Copy Markdown
Collaborator

No, it is fine, we don't follow a strict workflow, I thought we discussed this somewhere before... In #4359 actually!

Funnily enough, I started a similar PR in #4376 but kind of forgot about it. Could you check @soulomoon's comment from there?

@soulomoon

soulomoon commented Apr 4, 2025

Copy link
Copy Markdown
Collaborator

Yes, benchmark is in CI, if you wish to turn it off by default, you need to modify the CI as well to explicitily enable it.

@fendor fendor added the performance Issues about memory consumption, responsiveness, etc. label Apr 9, 2025
@fendor fendor added the status: needs review This PR is ready for review label Apr 9, 2025
@fendor

fendor commented Apr 10, 2025

Copy link
Copy Markdown
Collaborator

@soulomoon Looks like it worked? Does this look good then?

@soulomoon soulomoon left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Look good to me

@fendor
fendor enabled auto-merge (squash) April 13, 2025 12:01
@fendor
fendor merged commit 9127381 into haskell:master Apr 13, 2025
@juhp
juhp deleted the patch-2 branch May 9, 2025 05:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance Issues about memory consumption, responsiveness, etc. status: needs review This PR is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add Cabal flag for building ghcide-bench

3 participants