Skip to content

Conversation

@fisx
Copy link
Contributor

@fisx fisx commented Jan 2, 2026

Also force people to have cabal.project.local, since some of the flags are important for dev machines to work smoothly.

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

Also force people to have cabal.project.local, since some of the flags
are important for dev machines to work smoothly.
@fisx fisx requested a review from a team as a code owner January 2, 2026 15:07
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Jan 2, 2026
@fisx fisx requested review from blackheaven and removed request for blackheaven January 2, 2026 15:08

.PHONY: c
c-fast:
if [ ! -e "cabal.project.local" ]; then \
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm wondering if you're re-inventing the cabal.project.local Makefile target here? 🤔

IMHO there should be at least a comment which explains how these two ways relate and why yours is needed/better/... 😉

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oops, indeed, forgot about that. i'll try again tomorrow.

@fisx fisx requested a review from supersven January 2, 2026 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants