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

Apply GitHub style to exporter matrix #8993

Merged
merged 2 commits into from
Dec 10, 2018
Merged

Conversation

yennster
Copy link
Contributor

@yennster yennster commented Dec 6, 2018

Description

This PR changes the table output produced by mbed export -S from this...
screen shot 2018-12-06 at 10 38 03 am
To this...
screen shot 2018-12-06 at 10 37 30 am

Which allows simple copy/paste in github style table markdown (similar to PR #7720), for example...

Platform cces cmake_gcc_arm codeblocks e2studio eclipse_armc5 eclipse_gcc_arm eclipse_iar embitz gnuarmeclipse iar make_armc5 make_armc6 make_gcc_arm make_iar mcuxpresso netbeans qtcreator sw4stm32 uvision5 uvision6 vscode_armc5 vscode_gcc_arm vscode_iar
ARCH_BLE - x - x x x - - x - x - x - - x x - x - x x -
ARCH_BLE_BOOT - x - x x x - - x - x - x - - x x - x - x x -
ARCH_BLE_OTA - x - x x x - - x - x - x - - x x - x - x x -

And it just looks cleaner and is easier to read.

I also added another "\n" blank space in between the end of the table and the table information (total IDEs, platforms, etc.)

Pull request type

[ ] Fix
[ ] Refactor
[ ] Target update
[X] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

@cmonr
Copy link
Contributor

cmonr commented Dec 6, 2018

@yennster Could you rebase your PR?
I don't know why, but Travis CI is being weird and not updating itself correctly.

@cmonr
Copy link
Contributor

cmonr commented Dec 10, 2018

CI started

@mbed-ci
Copy link

mbed-ci commented Dec 10, 2018

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 1
Build artifacts

@cmonr cmonr merged commit 3df9421 into ARMmbed:master Dec 10, 2018
@cmonr cmonr removed the needs: CI label Dec 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants