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

Removing old filesystem test #4129

Merged
merged 1 commit into from Apr 20, 2017

Conversation

Projects
None yet
6 participants
@bridadan
Contributor

bridadan commented Apr 6, 2017

Description

This should solve #4114. This PR removes an old Filesystem mbed 2 test. The source directory for the test doesn't exist anymore, so we should probably remove the test listing!

Status

READY

Migrations

If this PR changes any APIs or behaviors, give a short description of what API users should do when this PR is merged.

NO

Todos

  • mbed 2 test

FYI @pranavshandilya, @geky

@bridadan bridadan requested review from theotherjimmy and 0xc0170 Apr 6, 2017

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Apr 6, 2017

should be "Remove old filesystem test" but whatever.

@theotherjimmy

Yes please.

@geky

geky approved these changes Apr 6, 2017

👍

@bridadan

This comment has been minimized.

Contributor

bridadan commented Apr 6, 2017

@mbed-bot: TEST

HOST_OSES=ALL
BUILD_TOOLCHAINS=ALL
TARGETS=ALL

@bridadan bridadan added the needs: CI label Apr 6, 2017

@mbed-bot

This comment has been minimized.

mbed-bot commented Apr 6, 2017

[Build 1338]
SUCCESS: Building succeeded and tests were run! Be sure to check the test results

@bridadan bridadan added needs: review and removed needs: CI labels Apr 7, 2017

@adbridge adbridge merged commit 29d59ed into ARMmbed:master Apr 20, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/mbedci Merged build finished. No test results found.
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