Skip to content

Conversation

saheerb
Copy link
Contributor

@saheerb saheerb commented Sep 6, 2021

No description provided.

@saheerb saheerb requested a review from donatieng September 6, 2021 08:41
Copy link
Contributor

@donatieng donatieng left a comment

Choose a reason for hiding this comment

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

Thanks Saheer, one minor nit below:

@jeromecoutant
Copy link

Need to wait for updates in mbed-os repo ?

Co-authored-by: Donatien Garnier <donatien.garnier@arm.com>
Copy link
Contributor

@donatieng donatieng left a comment

Choose a reason for hiding this comment

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

@ChrisGooch-Arm let's wait for the above issue and PR to be in first, they we can merge this one

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 6, 2021

Both of above will be most likely resolved within today, I'll keep an eye on both.

@ChrisGooch-Arm
Copy link
Collaborator

@donatieng, @0xc0170 - is this ok to merge now or is there still a dependency on other things?

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 5, 2021

Yes, please lets merge. All was resolved for the Mbed OS release in September.

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 21, 2021

@ChrisGooch-Arm can we merge this, its needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

5 participants