-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Fixing warnings in RIOT static test #22216
Copy link
Copy link
Open
Labels
Area: CIArea: Continuous Integration of RIOT componentsArea: Continuous Integration of RIOT componentsType: bugThe issue reports a bug / The PR fixes a bug (including spelling errors)The issue reports a bug / The PR fixes a bug (including spelling errors)Type: trackingThe issue tracks and organizes the sub-tasks of a larger effortThe issue tracks and organizes the sub-tasks of a larger effort
Description
Metadata
Metadata
Assignees
Labels
Area: CIArea: Continuous Integration of RIOT componentsArea: Continuous Integration of RIOT componentsType: bugThe issue reports a bug / The PR fixes a bug (including spelling errors)The issue reports a bug / The PR fixes a bug (including spelling errors)Type: trackingThe issue tracks and organizes the sub-tasks of a larger effortThe issue tracks and organizes the sub-tasks of a larger effort
Type
Fields
Give feedbackNo fields configured for Task.
Description
RIOT CI currently executes multiple tests before code could be merged to master. PR #22183 fixes 10 warning which are clearly visible, but further investigation of this issues during PR #22186 shows that there are many more warnings, probably in older files before CI was enabled. This issue is an attempt to track fixes of this warning.
Fixed directories
This issue track directories which are reviewed and static tests warning are fixed.
Testing procedure
Run locally
make static-testor look at static-test in Github Actions for master branch and check if number of warnings from static tests in fixed directories was reduced.