Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add R&D flag feature checks for SD card script execution #573

Merged
merged 5 commits into from
Dec 1, 2023

Conversation

jlaitine
Copy link

No description provided.

@jlaitine jlaitine requested a review from t-salminen November 30, 2023 09:48
@jlaitine jlaitine force-pushed the add_app_to_check_rd_flags branch 2 times, most recently from bb2dfc8 to f939b82 Compare November 30, 2023 13:04
@jlaitine
Copy link
Author

Now it seems to be working, ready for review ...

@jlaitine
Copy link
Author

Oops, still need to fix the pixhawk and sitl builds...

@jlaitine jlaitine force-pushed the add_app_to_check_rd_flags branch 2 times, most recently from 34611d4 to e2e4991 Compare November 30, 2023 13:39
@jlaitine
Copy link
Author

Now all should be good again

@jlaitine jlaitine force-pushed the add_app_to_check_rd_flags branch from e2e4991 to d794927 Compare November 30, 2023 14:04
Copy link

@pussuw pussuw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Add a systemcmd to check R&D feature flags for the board

Signed-off-by: Jukka Laitinen <[email protected]>
Place them inside the board configuration instead

Signed-off-by: Jukka Laitinen <[email protected]>
@jlaitine jlaitine force-pushed the add_app_to_check_rd_flags branch from d794927 to bd92902 Compare December 1, 2023 07:25
@jlaitine
Copy link
Author

jlaitine commented Dec 1, 2023

Updated submodules & merging

@jlaitine jlaitine merged commit 3bbe540 into main Dec 1, 2023
14 of 15 checks passed
@jlaitine jlaitine deleted the add_app_to_check_rd_flags branch December 1, 2023 08:01
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