Skip to content

Add option to enable/disable warning as error - #1293

Closed
SunBlack wants to merge 1 commit into
BehaviorTree:masterfrom
SunBlack:BTCPP_ENABLE_WARNING_AS_ERROR
Closed

SunBlack wants to merge 1 commit into
BehaviorTree:masterfrom
SunBlack:BTCPP_ENABLE_WARNING_AS_ERROR

Conversation

@SunBlack

Copy link
Copy Markdown
Contributor

Adding this option to be able to bypass issues like this: #1292

@facontidavide

Copy link
Copy Markdown
Collaborator

why?

@SunBlack

Copy link
Copy Markdown
Contributor Author

why?

For example, if a new compiler version is released that introduces a new warning, you can no longer build that version here until a new version has been released. The same applies to compilers that are not tested here by CI.

@facontidavide

Copy link
Copy Markdown
Collaborator

in that case, you should submit a fix of the new error and add the compile to the CI matric.

disabling it is not an option

@SunBlack
SunBlack deleted the BTCPP_ENABLE_WARNING_AS_ERROR branch November 13, 2026 20:54
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.

2 participants