-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Cleanup get_config (mbed compile --config) #6580
Conversation
Diff between how it was and it is now part of this PR? |
@0xc0170 Yeah, just a sec. Updated. |
/morph build |
Build : SUCCESSBuild number : 1756 Triggering tests/morph test |
Exporter Build : SUCCESSBuild number : 1393 |
Test : SUCCESSBuild number : 1563 |
Still waiting on @MarceloSalazar and/or @screamerbg. |
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.
LGTM
Description
I made get_config a little nicer to read, and the output a little nicer
to read.
before
(Yes, it's all one line and contains config)
After
Each macro on a line, like the config section, and does not contain config because that's redundant.
Pull request type
[X] Fix
[ ] Refactor
[ ] New target
[ ] Feature
[ ] Breaking change