You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Right now, header imports are a friggin mess. Clean that stuff up.
Acceptance Criteria
Follow these header guidelines:
All things imported should be directly used in the same file
Import something into a .c file if you use the API but import to the .h file if you depend on public types
Use <> includes only for standard libraries (std* stuff); use "" for CubeMX dependencies, embedded-base, and our code isndie the project
Proposed Solution
Follow the acceptance criteria, and also it may just be easiest to remove each header until shit breaks. Make sure you arent employing transitive dependencies such that one file includes another file that includes the actual dependency. So if you remove a import from file A and file B breaks, then file B needs the import not file A.
The text was updated successfully, but these errors were encountered:
Description
Right now, header imports are a friggin mess. Clean that stuff up.
Acceptance Criteria
Follow these header guidelines:
<>
includes only for standard libraries (std* stuff); use""
for CubeMX dependencies, embedded-base, and our code isndie the projectProposed Solution
Follow the acceptance criteria, and also it may just be easiest to remove each header until shit breaks. Make sure you arent employing transitive dependencies such that one file includes another file that includes the actual dependency. So if you remove a import from file A and file B breaks, then file B needs the import not file A.
The text was updated successfully, but these errors were encountered: