Skip to content
Snippets Groups Projects

Comment out warnings

Merged Carsten Gräser requested to merge feature/remove-warning into master

These warnings are very irritating. They do not warn about deprecated or misused feature. They are issued any time you compile the tests. They don't provide any useful information unless you look at the code and have insight in the implementation detains.

To sum up: They don't look like a warning addressed to the user, but like a reminder on an intended future implementation addressed to the implementor. Hence they should not be exposed to the user.

If anyone feels that this is an important user warning feel free to re-enable it after adding enough information making the warning actually useful.

Merge request reports

Pipeline #4613 passed

Pipeline passed for b4299314 on feature/remove-warning

Merged by avatar (Apr 2, 2025 1:05am UTC)

Loading

Pipeline #4614 passed

Pipeline passed for 9c0eb7fd on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading