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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed superfluous space #4072

Merged
merged 1 commit into from Apr 6, 2017

Conversation

Projects
None yet
7 participants
@seppestas
Contributor

seppestas commented Mar 30, 2017

The extra space between "--no_wrap_diagnostics" and "-e" is inconsistent with the development and release targets.

It bugs people (like me) that have little OCD tics 馃槈.

Removed superfluous space
The extra space between "--no_wrap_diagnostics" and "-e" is inconsistent with the development and release targets.

It bugs people (like me) that have little OCD tics 馃槈.
@0xc0170

This comment has been minimized.

Member

0xc0170 commented Mar 30, 2017

/morph test

@0xc0170 0xc0170 added the needs: CI label Mar 30, 2017

@seppestas

This comment has been minimized.

Contributor

seppestas commented Mar 30, 2017

Hhm, not sure why this needs CI checks. It's just a little whitespace change...

@bulislaw

This comment has been minimized.

Member

bulislaw commented Mar 30, 2017

We need that to be able to merge PR.

@mbed-bot

This comment has been minimized.

mbed-bot commented Mar 30, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1795

All builds and test passed!

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Mar 30, 2017

retest uvisor

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Mar 31, 2017

@mazimkhan Please look at uvisor job for this patch

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Apr 3, 2017

retest uvisor

@bridadan bridadan added ready for merge and removed needs: CI labels Apr 4, 2017

@sg- sg- merged commit caded38 into ARMmbed:master Apr 6, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-test Job has completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment