Skip to content

Add -Werror under -fdevelopment - #302

Merged
riz0id merged 3 commits into
masterfrom
riz0id/development-flag-werror
Jan 27, 2026
Merged

Add -Werror under -fdevelopment#302
riz0id merged 3 commits into
masterfrom
riz0id/development-flag-werror

Conversation

@riz0id

@riz0id riz0id commented Jan 6, 2026

Copy link
Copy Markdown
Collaborator

Guards the -Werror flag under a -fdevelopment flag. The -Werror flag is incompatible with Hackage uploads.

@riz0id
riz0id requested a review from j6carey January 6, 2026 14:01
@riz0id riz0id self-assigned this Jan 6, 2026
@ncaq

ncaq commented Jan 6, 2026

Copy link
Copy Markdown
Contributor

I think another way.
We write cabal.project.
I do approch for -j too.

@riz0id

riz0id commented Jan 6, 2026

Copy link
Copy Markdown
Collaborator Author

I think another way. We write cabal.project'. I do approch for -j` too.

Are you suggesting that the option be changed via a cabal.project?

Comment thread proto3-suite.cabal Outdated
Comment thread proto3-suite.cabal Outdated

@j6carey j6carey 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.

See specific comments.

@ncaq

ncaq commented Jan 6, 2026

Copy link
Copy Markdown
Contributor

Are you suggesting that the option be changed via a cabal.project?

If you non need build flag, I want share info, I don't suggest.

@riz0id

riz0id commented Jan 7, 2026

Copy link
Copy Markdown
Collaborator Author

Are you suggesting that the option be changed via a cabal.project?

If you non need build flag, I want share info, I don't suggest.

Gotcha, well thank you! :)

@riz0id
riz0id requested a review from j6carey January 7, 2026 15:30
@riz0id
riz0id merged commit d5c3c96 into master Jan 27, 2026
48 checks passed
@riz0id
riz0id deleted the riz0id/development-flag-werror branch January 27, 2026 15:44
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.

3 participants