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

Add comment about local development #2772

Merged
merged 2 commits into from
Jan 14, 2023

Conversation

weebl2000
Copy link

No description provided.

README.rst Outdated Show resolved Hide resolved
Useful parameters to set in cabal.project
@weebl2000
Copy link
Author

@Jimbo4350 Sorry for tag, but not sure if Duncan is still involved in the project. Small comment about developing locally that might help people. Suggestions have been processed.

@weebl2000
Copy link
Author

@coot Sorry for shameless tagging. But this was basically approved. Can we merge it?

@coot coot requested a review from Jimbo4350 January 26, 2022 10:41
@newhoggy newhoggy requested a review from a team as a code owner January 7, 2023 02:13
@coot
Copy link
Contributor

coot commented Jan 11, 2023

I prefer the other way around, remove -Werror and only use it in CI and for those who want to have it in their dev cycle, set -Werror in cabal.project.local file.

@newhoggy newhoggy self-requested a review January 14, 2023 02:25
@newhoggy newhoggy merged commit 0a066fe into IntersectMBO:master Jan 14, 2023
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.

6 participants