-
Notifications
You must be signed in to change notification settings - Fork 3k
Travis: fix astyle report (-/+ files) #7898
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
AStyle only reports the total number (should be - few files). Master reports 553 files, here we got 529 files so negative number should be reported here |
1b3987d
to
88751a8
Compare
88751a8
to
6ece6aa
Compare
45ac626
to
a4065cb
Compare
Looks like Travis astyle report is fixed |
Ready for reviews ! |
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.
Good catches in the astyle script 👍
Will rebase after 5.10.1, and update it |
a4065cb
to
f1f16e5
Compare
Rescoped this PR. It now only fixes Travis AStyle report. I'll send the files separately in another PR @cmonr Please review |
/morph build |
Build : SUCCESSBuild number : 3300 Triggering tests/morph test |
ci-morph-mbed2-build failed because the ARM license server is down. xD ಠ_ಠ |
Arm License network issues, now fixed. /morph export-build |
/morph mbed2-build |
Exporter Build : SUCCESSBuild number : 2934 |
Test : SUCCESSBuild number : 3109 |
Description
Touching few files from various area, the changes are minimal.
Test for #7897 (to confirm this is the issue currently, recent PR shows only numbers so either the number has not changed or not working). This is important to make working , no more + number in the PR 👍
Pull request type