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

Nanostack v6.2.2 #5249

Merged
merged 1 commit into from Oct 5, 2017

Conversation

Projects
None yet
9 participants
@JuhPuur
Contributor

JuhPuur commented Oct 4, 2017

Description

Nanostack v6.2.2 patch release targeted for mbed-os-5.6.2
TCP protocol improvements

Status

READY

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Oct 4, 2017

Is commit "Nanostack v6.2.2" descriptive enough - no release notes (what this update contains)? or at least a link?

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Oct 4, 2017

who should review this? Please tag them for approval

@JuhPuur

This comment has been minimized.

Contributor

JuhPuur commented Oct 4, 2017

Doesn't look like I can add a reviewer.

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Oct 4, 2017

Doesn't look like I can add a reviewer.

just cc them here in the comment, they will be notificed and we wait for them to approve the PR

@JuhPuur

This comment has been minimized.

Contributor

JuhPuur commented Oct 4, 2017

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Oct 4, 2017

/morph test-nightly

@SeppoTakalo SeppoTakalo requested a review from mikter Oct 4, 2017

@SeppoTakalo

This comment has been minimized.

Contributor

SeppoTakalo commented Oct 4, 2017

@mikter As a Tech lead for Mesh, we need your +1 or -1 for this.

@mikaleppanen

This comment has been minimized.

Contributor

mikaleppanen commented Oct 4, 2017

Looks good to me.

@mikter

This comment has been minimized.

Contributor

mikter commented Oct 4, 2017

@artokin @deepakvenugopal you need to check these also

@mikter

mikter approved these changes Oct 4, 2017

@artokin

This comment has been minimized.

Contributor

artokin commented Oct 4, 2017

+1, nanostack-border-router tested OK with this PR.

@mbed-bot

This comment has been minimized.

mbed-bot commented Oct 4, 2017

Result: SUCCESS

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

/morph test-nightly

Output

mbed Build Number: 1522

All builds and test passed!

@0xc0170 0xc0170 added ready for merge and removed needs: CI labels Oct 5, 2017

@theotherjimmy theotherjimmy merged commit f47d8e9 into ARMmbed:master Oct 5, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-test-nightly 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