-
Notifications
You must be signed in to change notification settings - Fork 3k
Move target related code into it's own directory #3995
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
Conversation
This should make it easier to navigate the code base
/morph test |
@0xc0170 After this is merged, we should push partners to put there target code here. I moved all of it that's currently in the tools. I just want new target code to go here too. |
@c1728p9 Part of the technical debt I was talking about earlier. |
@bridadan Any more tests to run? |
@screamerbg Could we mention the owners of the moved target code in this PR to get their input on the move? in particular this PR affects the location of LPC and NCS target-specific files. |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
OutputAll builds and test passed! |
Looks good, thanks for notifying Realtek. |
+1, will do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Loving the changes! We should run the mbed 2 bot too just to double check
@mbed-bot: TEST HOST_OSES=ALL |
Like it! LGTM |
[Build 1314] |
This is failing due to the issue I mention here: #4021 (comment) We'll need help from Nordic to fix the issue, stay tuned over on #4021 |
@mbed-bot: TEST HOST_OSES=ALL |
[Build 1325] |
This should make it easier to navigate the code base, and reduces
technical debt.
Testing