Skip to content

events: astyle fix #7691

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 7, 2018
Merged

events: astyle fix #7691

merged 1 commit into from
Aug 7, 2018

Conversation

0xc0170
Copy link
Contributor

@0xc0170 0xc0170 commented Aug 3, 2018

Description

AStyle fix for events folder

Pull request type

[X] Fix
[ ] Refactor
[ ] New target
[ ] Feature
[ ] Breaking change

@0xc0170 0xc0170 requested a review from a team August 3, 2018 12:24
NirSonnenschein
NirSonnenschein approved these changes Aug 6, 2018
View reviewed changes
Copy link
Contributor

@NirSonnenschein NirSonnenschein left a comment

Choose a reason for hiding this comment

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

looks ok to me (but it is a very large PR, so I may heavy missed something)

geky
geky approved these changes Aug 6, 2018
View reviewed changes
Copy link
Contributor

@geky geky left a comment

Choose a reason for hiding this comment

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

Yeah, idunno, looks fine to me.

@cmonr
Copy link
Contributor

cmonr commented Aug 6, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Aug 6, 2018

Build : SUCCESS

Build number : 2748
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/7691/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Aug 6, 2018

Exporter Build : SUCCESS

Build number : 2379
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/exporter/7691/

@mbed-ci
Copy link

mbed-ci commented Aug 7, 2018

Test : FAILURE

Build number : 2472
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/7691/2472

@NirSonnenschein
Copy link
Contributor

@0xc0170 a strange error came up in the test scripts. may be unrelated to the PR. does it look familiar to you?

@0xc0170
Copy link
Contributor Author

0xc0170 commented Aug 7, 2018

/morph test
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Aug 7, 2018

Test : SUCCESS

Build number : 2473
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/7691/2473

@cmonr cmonr merged commit 273a052 into ARMmbed:master Aug 7, 2018
@andresag01 andresag01 mentioned this pull request Aug 7, 2018
Closed
pan- pushed a commit to pan-/mbed that referenced this pull request Aug 22, 2018
Merge pull request ARMmbed#7691 from 0xc0170/fix_events_style
ca588c4 
events: astyle fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cmonr cmonr cmonr approved these changes

@geky geky geky approved these changes

@NirSonnenschein NirSonnenschein NirSonnenschein approved these changes

Assignees
No one assigned
Labels
release-version: 5.9.5
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@0xc0170 @cmonr @mbed-ci @NirSonnenschein @geky