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

TRAVIS: turn off upload python test code coverage #10524

Merged
merged 1 commit into from
May 7, 2019

Conversation

jamesbeyond
Copy link
Contributor

Description

Turn off upload python test code coverage from TRAVIS.
We are planing to upload C/CPP code coverage to coverall.io later on.

Pull request type

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

Reviewers

@cmonr

@jamesbeyond jamesbeyond changed the title TRAVIS: trun off upload python test code coverage TRAVIS: turn off upload python test code coverage May 2, 2019
@0xc0170
Copy link
Contributor

0xc0170 commented May 3, 2019

Ci started

@0xc0170
Copy link
Contributor

0xc0170 commented May 3, 2019

@jamesbeyond thinking about the release target for this. I am inclined for 5.13.

One question - this python test coverage goes to tools repository (so can be removed here and wont lose this test coverage) ?

@jamesbeyond
Copy link
Contributor Author

Hi @0xc0170,
The python code coverage is not for https://github.com/ARMmbed/mbed-os-tools , it is for the tools folder inside mbed-os. the Travis still run the test and generate coverage, but it is just not uploading them, but we are not loosing anything. if there is some mirror repos we can upload to there.

Turn this python code coverage off is for uploading the C/CPP code coverage. which is more useful. I don't think we need to wait for 5.13. because it is not a function change. will not have impact to anyone.
Surely it can be announced in 5.13, let people aware of that.

@adbridge adbridge merged commit 42a9a7a into ARMmbed:master May 7, 2019
@jamesbeyond jamesbeyond deleted the travis branch November 6, 2019 11:18
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.

4 participants